-
Notifications
You must be signed in to change notification settings - Fork 75
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
move go.mod to project root #114
base: main
Are you sure you want to change the base?
Conversation
4410e8f
to
6553c1d
Compare
Signed-off-by: Matthieu MOREL <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for trying to fix this.
I'm not sure if it works for the people using go, but just wanted to highlight one thing about the locations of some files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will it be possible to keep all those changes in the "go/" directory?
The reason is, I think it would be best to keep these files separated from the proto sources.
I'm not sure what causes these, but would like all the "go" related stuff that don't use bazel to be in their own directory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not with the actual capabilities of bazel_gazelle, see https://dev.azure.com/cncf/xds/_build/results?buildId=182921&view=logs&j=6614165d-8168-5766-9db6-f35f537f8e36&t=357c2333-0bbe-5588-5320-1bd2a4987c75&l=52, the name of the file is checked said here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this pattern should be discouraged.
IMHO having the go files so it is easier to use them is great, but it should be limited to a specific sub-dir that is different than where the "source files" (protos) are.
This moves go.mod to the root directory of the project and the go code follows this change to.
Closes #104