Skip to content

bazel: add more detail to dev test cobra msg#82482

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
msbutler:butler-dev-msg
Jun 8, 2022
Merged

bazel: add more detail to dev test cobra msg#82482
craig[bot] merged 1 commit intocockroachdb:masterfrom
msbutler:butler-dev-msg

Conversation

@msbutler
Copy link
Copy Markdown
Collaborator

@msbutler msbutler commented Jun 6, 2022

Fixes #82411

Release note: None

@msbutler msbutler requested a review from irfansharif June 6, 2022 19:21
@msbutler msbutler self-assigned this Jun 6, 2022
@msbutler msbutler requested a review from a team as a code owner June 6, 2022 19:21
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@msbutler
Copy link
Copy Markdown
Collaborator Author

msbutler commented Jun 6, 2022

If I rerun ./dev build short, the cobra message does not reflect my changes. What's the easiest way to rebuild the dev package to inspect my changes?

@rickystewart
Copy link
Copy Markdown
Collaborator

@msbutler dev build dev

Copy link
Copy Markdown
Collaborator

@rickystewart rickystewart left a comment

Choose a reason for hiding this comment

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

Nit: detail in the commit message, not detial.

@msbutler msbutler changed the title bazel: add more detial to dev test cobra msg bazel: add more detail to dev test cobra msg Jun 6, 2022
Copy link
Copy Markdown
Contributor

@irfansharif irfansharif left a comment

Choose a reason for hiding this comment

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

LGTM, appended a cosmetic commit to unify how the help text looks across dev test -h and dev bench -h -- feel free to drop it.

@msbutler msbutler force-pushed the butler-dev-msg branch 2 times, most recently from b0af956 to 46670b0 Compare June 8, 2022 13:59
Copy link
Copy Markdown
Contributor

@irfansharif irfansharif left a comment

Choose a reason for hiding this comment

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

Thanks Michael, this will really help!

@msbutler
Copy link
Copy Markdown
Collaborator Author

msbutler commented Jun 8, 2022

bors r=irfansharif, rickystewart

@craig craig bot merged commit 7d365bc into cockroachdb:master Jun 8, 2022
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jun 8, 2022

Build succeeded:

@msbutler msbutler deleted the butler-dev-msg branch June 9, 2022 02:51
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.

bazel: add more detail to ./dev test --help

4 participants