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

Feature/add additional value #29

Merged
merged 16 commits into from
Oct 15, 2021
Merged

Feature/add additional value #29

merged 16 commits into from
Oct 15, 2021

Conversation

ktkfree
Copy link
Contributor

@ktkfree ktkfree commented Oct 13, 2021

** 연관 이슈**
siiesissue-10

** 변경 내용 **
. keycloak Create, Read 구현
. CSP type 추가

@zugwan
Copy link
Contributor

zugwan commented Oct 15, 2021

Go 모듈 중 tks-proto 버전이 업데이트 되지 않아서 빌드 오류가 발생합니다. 아래처럼 변경이 되어야 합니다.

-       github.com/openinfradev/tks-proto v0.0.6-0.20210924020717-178698d59e9d
+       github.com/openinfradev/tks-proto v0.0.6-0.20211015003551-ed8f9541f40d

@ktkfree
Copy link
Contributor Author

ktkfree commented Oct 15, 2021

Go 모듈 중 tks-proto 버전이 업데이트 되지 않아서 빌드 오류가 발생합니다. 아래처럼 변경이 되어야 합니다.

-       github.com/openinfradev/tks-proto v0.0.6-0.20210924020717-178698d59e9d
+       github.com/openinfradev/tks-proto v0.0.6-0.20211015003551-ed8f9541f40d

네 맞습니다.
저희 tks 는 서비스간 정적인 링크가 많아, 동시에 여러 component 를 변경할 경우 PR 의 에로사항이 있네요...
go.mod 가 자동으로 main branch 를 업데이트하는 방식이 아니라 버전을 명시해야 하는 구조인것 같습니다.
말씀하신대로 각 서비스간 PR 이 완료되는대로 하나씩 모든 go.mod 파일의 업데이트를 하려고 하고 있는데, 혹시 좋은 방법이 있을까요?

@ktkfree ktkfree merged commit 48bc6cf into main Oct 15, 2021
@ktkfree ktkfree deleted the feature/add_additional_value branch October 15, 2021 08:32
@zugwan
Copy link
Contributor

zugwan commented Oct 18, 2021

Go 모듈 중 tks-proto 버전이 업데이트 되지 않아서 빌드 오류가 발생합니다. 아래처럼 변경이 되어야 합니다.

-       github.com/openinfradev/tks-proto v0.0.6-0.20210924020717-178698d59e9d
+       github.com/openinfradev/tks-proto v0.0.6-0.20211015003551-ed8f9541f40d

네 맞습니다. 저희 tks 는 서비스간 정적인 링크가 많아, 동시에 여러 component 를 변경할 경우 PR 의 에로사항이 있네요... go.mod 가 자동으로 main branch 를 업데이트하는 방식이 아니라 버전을 명시해야 하는 구조인것 같습니다. 말씀하신대로 각 서비스간 PR 이 완료되는대로 하나씩 모든 go.mod 파일의 업데이트를 하려고 하고 있는데, 혹시 좋은 방법이 있을까요?

저도 golang 개발할 때 모범 사례가 무엇인지 잘 모르겠지만 현 시점에서는 PR을 등록하는 시점에 다시 확인하거나 지금처럼 github action에서 빌드 실패 이후 수정하는 방법 밖에 잘 생각이 나지 않네요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants