Skip to content

build: add instructions to rectify code that is not regenerated#56004

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
otan-cockroach:lint_change
Oct 27, 2020
Merged

build: add instructions to rectify code that is not regenerated#56004
craig[bot] merged 1 commit intocockroachdb:masterfrom
otan-cockroach:lint_change

Conversation

@otan
Copy link
Copy Markdown
Contributor

@otan otan commented Oct 26, 2020

Make it easier for the developer to know how to fix un-generated code by
specifying the required commands.

Release note: None

@otan otan requested a review from jlinder October 26, 2020 23:51
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@otan
Copy link
Copy Markdown
Contributor Author

otan commented Oct 26, 2020

(failure example: #56005)

run build/builder.sh make generate &> artifacts/generate.log || (cat artifacts/generate.log && false)
run build/builder.sh make buildshort &> artifacts/buildshort.log || (cat artifacts/buildshort.log && false)
rm artifacts/generate.log
check_clean "Run \`make generate\` to automatically fix these. For only BUILD.bazel issues, run \`bazel run //:gazelle\`."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd also be fine with suggesting make bazel-generate here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

have to teach people to stop using make somehow :P

@otan otan force-pushed the lint_change branch 6 times, most recently from ead0b9d to 0faa730 Compare October 27, 2020 14:18
Make it easier for the developer to know how to fix un-generated code by
specifying the required commands.

Release note: None
@otan
Copy link
Copy Markdown
Contributor Author

otan commented Oct 27, 2020

bors r=irfansharif

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 27, 2020

Build succeeded:

@craig craig bot merged commit d058e01 into cockroachdb:master Oct 27, 2020
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