Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented May 11, 2019

This PR will move all structs maintaining APIs needed from code.gitea.io/sdk-go to this main repository's sub package code.gitea.io/gitea/modules/structs. That will make maintaining APIs easier. When creating a new API for gitea, an extra PR to code.gitea.io/sdk-go will not be a necessary.

@lunny lunny added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label May 11, 2019
@lunny lunny added this to the 1.9.0 milestone May 11, 2019
Copy link
Member

@techknowlogick techknowlogick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thanks for PR :)

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label May 11, 2019
@techknowlogick
Copy link
Member

CI Fail

@techknowlogick
Copy link
Member

needs make fmt

@techknowlogick
Copy link
Member

needs make vendor to cleanup vendor/modules.txt

@codecov-io
Copy link

codecov-io commented May 11, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@1658cd0). Click here to learn what that means.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #6905   +/-   ##
=========================================
  Coverage          ?   41.39%           
=========================================
  Files             ?      436           
  Lines             ?    59618           
  Branches          ?        0           
=========================================
  Hits              ?    24678           
  Misses            ?    31700           
  Partials          ?     3240
Impacted Files Coverage Δ
routers/api/v1/org/hook.go 0% <ø> (ø)
routers/api/v1/org/member.go 12.13% <ø> (ø)
routers/api/v1/utils/hook.go 0% <ø> (ø)
models/attachment.go 52.05% <ø> (ø)
models/issue_tracked_time.go 67.56% <ø> (ø)
modules/structs/utils.go 0% <ø> (ø)
routers/api/v1/repo/file.go 68.19% <ø> (ø)
models/webhook_msteams.go 0% <ø> (ø)
routers/api/v1/repo/issue_comment.go 54.51% <ø> (ø)
models/lfs_lock.go 63.63% <ø> (ø)
... and 68 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1658cd0...72046eb. Read the comment docs.

@zeripath
Copy link
Contributor

Hmm 2 things:

  • Would it actually be better to call modules/structs modules/api?
  • Is the SDK just going to import these from its code or will it keep its own copy?

@zeripath zeripath changed the title Move sdk structs to moduels/structs Move sdk structs to modules/structs May 11, 2019
@lunny
Copy link
Member Author

lunny commented May 11, 2019

@zeripath
First, It only includes structs but not any codes we called API, so I don't think we should call it API.
Second, Yes, after this PR merged I will send another PR to code.gitea.io/sdk-go to remove all structs there and depend on this new package's specify version. That means you have to use go mod build code.gitea.io/sdk-go because code.gitea.io/gitea/modules/structs may have broken changes.

@zeripath
Copy link
Contributor

Ok.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels May 11, 2019
@lunny lunny merged commit 34eee25 into go-gitea:master May 11, 2019
@lunny lunny deleted the lunny/move_sdk_structs branch May 11, 2019 10:21
zeripath added a commit to zeripath/gitea that referenced this pull request May 11, 2019
* move sdk structs to moduels/structs

* fix tests

* fix fmt

* fix swagger

* fix vendor
@lunny
Copy link
Member Author

lunny commented May 11, 2019

Should fixed #5954

techknowlogick pushed a commit that referenced this pull request May 11, 2019
* Add options to git.Clone to make it more capable

* Begin the process of removing the local copy and tidy up

* Remove Wiki LocalCopy Checkouts

* Remove the last LocalRepo helpers

* Remove WithTemporaryFile

* Enable push-hooks for these routes

* Ensure tests cope with hooks

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Remove Repository.LocalCopyPath()

* Move temporary repo to use the standard temporary path

* Fix the tests

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Remove LocalWikiPath

* Fix missing remove

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Use AppURL for Oauth user link (#6894)

* Use AppURL for Oauth user link

Fix #6843

* Update oauth.go

* Update oauth.go

* internal/ssh: ignore env command totally (#6825)

* ssh: ignore env command totally

* Remove commented code 

Needed fix described in issue #6889

* Escape the commit message on issues update and title in telegram hook (#6901)

* update sdk to latest (#6903)

* improve description of branch protection (fix #6886) (#6906)

The branch protection description text were not quite accurate.

* Fix logging documentation (#6904)

* ENABLE_MACARON_REDIRECT should be REDIRECT_MACARON_LOG

* Allow DISABLE_ROUTER_LOG to be set in the [log] section

* [skip ci] Updated translations via Crowdin

* Move sdk structs to modules/structs (#6905)

* move sdk structs to moduels/structs

* fix tests

* fix fmt

* fix swagger

* fix vendor
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/refactoring Existing code has been cleaned up. There should be no new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants