Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update swag version #750

Merged
merged 1 commit into from
Sep 16, 2021
Merged

Update swag version #750

merged 1 commit into from
Sep 16, 2021

Conversation

atg0831
Copy link
Member

@atg0831 atg0831 commented Sep 16, 2021

fix #744

Update swag version

  • before - v1.7.0
  • after - v1.7.1

#742 작업을 진행할 때 swag를 go install github.com/swaggo/swag/cmd/swag@latest로 설치를 해서
go.mod에 swag version이 업데이트가 안 되었습니다.
그래서 이번에는 go get github.com/swaggo/swag/cmd/swag 로 swag를 설치하면서 자동으로 go.mod의
swag version을 v1.7.1로 업데이트 하였습니다. 이 과정에서 go.sum 에서 추가적으로 변경이 되는 사항이 있는데
혹시나 dependency 관리에 문제가 될 사항은 없는지 확인 부탁드립니다.

swag version
before
- v1.7.0

after
- v1.7.1
@jihoon-seo
Copy link
Member

/lgtm
@atg0831
Build check 이 성공한 것을 보니, go.sum 변화도 문제가 없어 보입니다. 😊
기여 감사합니다! 😊

@github-actions github-actions bot added the lgtm This PR is acceptable by at least one reviewer label Sep 16, 2021
@seokho-son
Copy link
Member

감사합니다~~!! :)

@seokho-son
Copy link
Member

/approve

@jihoon-seo
Copy link
Member

jihoon-seo commented Sep 16, 2021

For anyone's information:

사실..
TB 메인 코드에서 github.com/swaggo/swag 패키지나 github.com/swaggo/swag/cmd/swag 패키지를 직접 import하여 사용하고 있는 것은 아니기 때문에
go.modv1.7.1 로 적혀 있어도
추후 다른 기여자가 swag 바이너리를 설치하기 위해

  • go install github.com/swaggo/swag/cmd/swag@latest
  • go get github.com/swaggo/swag/cmd/swag

두 명령 중 어떤 것을 실행해도
go.mod 에 적혀 있는 내용을 참조하여 v1.7.1 버전을 다운로드 하는 것이 아니라
실행 시점의 최신 버전을 다운로드하고 설치(=.../go/bin/ 같은 곳에 바이너리를 복사함)합니다.

따라서 go.modv1.7.1 버전을 명시한다고 해서
추후 다른 기여자가 swag 바이너리를 설치할 때 v1.7.1 버전이 설치되는 것은 아니지만

기여자에 따라서는
go.mod 파일을 열어 보고
"swag v1.7.1 이 사용되었었구나" 라는 사실을 참고할 수는 있을 것 같습니다. 😊

@seokho-son seokho-son merged commit ae6000d into cloud-barista:main Sep 16, 2021
@atg0831
Copy link
Member Author

atg0831 commented Sep 16, 2021

  • go install github.com/swaggo/swag/cmd/swag@latest
  • go get github.com/swaggo/swag/cmd/swag

두 명령 중 어떤 것을 실행해도
go.mod 에 적혀 있는 내용을 참조하여 v1.7.1 버전을 다운로드 하는 것이 아니라
실행 시점의 최신 버전을 다운로드하고 설치(=.../go/bin/ 같은 곳에 바이너리를 복사함)합니다.

궁금했던 부분이었는데 설명해 주셔서 이해되었습니다. 감사합니다.😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR is acceptable by at least one reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update swag version from v1.7.0 to v1.7.1 in go.mod
3 participants