Add GetRepoByID #511

						
				@ -160,0 +161,4 @@

					log.Println("== TestGetRepoByID ==")

					c := newTestClient()

					testrepo, _ := createTestRepo(t, "TestGetRepoByID", c)

6543 commented

2021-04-03 12:41:15 +00:00

-	
+

```diff - + ```

6543 marked this conversation as resolved

gitea/repo_test.go Outdated

						
				@ -160,0 +164,4 @@

					repo, _, err := c.GetRepoByID(testrepo.ID)

					assert.NoError(t, err)

					assert.NotNil(t, repo)

6543 commented

2021-04-03 12:41:37 +00:00

can you add assert.EqualValues(t, testrepo.ID, repo.ID) - just to make sure it do what is expected

@J0Nes90

can you add `assert.EqualValues(t, testrepo.ID, repo.ID)` - just to make sure it do what is expected @J0Nes90

6543 marked this conversation as resolved

6543 commented

2021-04-03 12:42:33 +00:00

@J0Nes90 just the two nits ☝️ (first one can be done with make fmt, i guess you just forgot it before commiting)

@J0Nes90 just the two nits ☝️ (first one can be done with `make fmt`, i guess you just forgot it before commiting)

J0Nes90 force-pushed J0Nes90/get_repo_by_id from a59f823ec5 to a711ae563f

2021-04-03 13:06:40 +00:00

Compare

J0Nes90 commented

2021-04-03 13:06:57 +00:00

fixed & force pushed

6543 approved these changes 2021-04-03 16:11:57 +00:00

Dismissed

lunny approved these changes 2021-04-03 16:24:41 +00:00

Dismissed

6543 merged commit 4e4d6dd731 into master

2021-04-03 16:35:19 +00:00

6543 referenced this issue from a commit

2021-04-03 16:35:19 +00:00

Add GetRepoByID (#511)

Sign in to join this conversation.

No reviewers

No Milestone

No Assignees

3 Participants

Notifications

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: gitea/go-sdk#511