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
Chore: Update Makefile to support go workspace #83549
Conversation
0ed4645
to
20b6547
Compare
20b6547
to
c900657
Compare
c900657
to
60ccc41
Compare
@@ -96,6 +96,7 @@ def pr_pipelines(): | |||
test_backend( | |||
get_pr_trigger( | |||
include_paths = [ | |||
"Makefile", |
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.
changing the makefile didn't trigger go tests
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.
Is that a concern?
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.
Oh I see, you already addressed it.
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.
yeah, sorry, the way i worded that wasn't clear
60ccc41
to
34508b0
Compare
What is this feature?
Adds support for the multi-module workspace in
make test-go
and make lint-go` targets.Why do we need this feature?
Sub packages are skipped when using
...
. See golang/go#50745 for more info.Who is this feature for?
[Add information on what kind of user the feature is for.]
Which issue(s) does this PR fix?:
Fixes #
Special notes for your reviewer:
Please check that: