lunny/gitea

Archived

[API] Add repoGetTag #16166

Merged

6543 merged 12 commits from api_repoGetTag into main

2021-06-23 21:08:48 +00:00

6543 commented

2021-06-15 22:20:26 +00:00

(Migrated from github.com)

GET /repos/{owner}/{repo}/tags/{tag} to get a single tag by name

`GET /repos/{owner}/{repo}/tags/{tag}` to get a single tag by name

zeripath reviewed 2021-06-18 05:36:54 +00:00

routers/api/v1/repo/tag.go

zeripath commented

2021-06-18 05:36:53 +00:00

Contributor

You're not returning empty here

You're not returning empty here

6543 (Migrated from github.com) reviewed 2021-06-18 07:43:46 +00:00

routers/api/v1/repo/tag.go

6543 (Migrated from github.com) commented

2021-06-18 07:43:46 +00:00

No i return 200

No i return 200

6543 (Migrated from github.com) reviewed 2021-06-20 14:52:42 +00:00

routers/api/v1/repo/tag.go

6543 (Migrated from github.com) commented

2021-06-20 14:52:42 +00:00

@zeripath DeleteTag codeblog got moved ...

@zeripath DeleteTag codeblog got moved ...

zeripath reviewed 2021-06-20 20:10:45 +00:00

routers/api/v1/repo/tag.go

zeripath commented

2021-06-20 20:10:45 +00:00

Contributor

Look at the line below (L138) you tell swagger that you're returning /resposnses/empty

Look at the line below (L138) you tell swagger that you're returning /resposnses/empty

6543 (Migrated from github.com) reviewed 2021-06-20 22:40:03 +00:00

routers/api/v1/repo/tag.go

6543 (Migrated from github.com) commented

2021-06-20 22:40:02 +00:00

haha right - good catch

haha right - good catch

zeripath approved these changes 2021-06-21 03:38:22 +00:00

techknowlogick reviewed 2021-06-23 19:11:24 +00:00

integrations/api_repo_tags_test.go

						
				@ -54,0 +62,4 @@

					// delete tag

					delReq := NewRequestf(t, "DELETE", "/api/v1/repos/%s/%s/tags/%s?token=%s", user.Name, repoName, newTag.Name, token)

					resp = session.MakeRequest(t, delReq, http.StatusNoContent)

techknowlogick commented

2021-06-23 19:11:24 +00:00

Contributor

Can you also add a test for a somewhat complicated tag, such as gitea/123?

Can you also add a test for a somewhat complicated tag, such as `gitea/123`?

techknowlogick approved these changes 2021-06-23 20:55:39 +00:00

This repo is archived. You cannot comment on pull requests.

No reviewers

No Label

good first issue

hacktoberfest-accepted

kind/deployment

kind/deprecated

kind/enhancement

kind/moderation

kind/regression

kind/translation

kind/upstream-related

performance/bigrepo

performance/cpu

performance/memory

performance/speed

priority/critical

priority/medium

proposal/rejected

reviewed/confirmed

reviewed/duplicate

reviewed/invalid

reviewed/not-a-bug

reviewed/wontfix

status/needs-feedback

theme/authentication

theme/backup-restore

theme/federation

theme/migration

theme/timetracker

No Milestone

No project

No Assignees

3 Participants

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: lunny/gitea#16166

No description provided.