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

Add support for bash completion of --config= names #10717

Closed
wants to merge 1 commit into from

Conversation

@elklein
Copy link
Contributor

elklein commented Feb 5, 2020

This change supports the following command line arguments:
--bazelrc=
--nosystem_rc
--noworkspace_rc
--nohome_rc

And supports finding bazelrc files in all the documented locations
as-per the docs at:
https://docs.bazel.build/versions/2.0.0/guide.html#bazelrc

It can handle command inheritance. So if you specify a "test" command
it knows to look at test, build, and common configs.

It also can deal with import and try-import commands, and can chase
those down to any nesting level.

This change supports the following command line arguments:
--bazelrc=<file>
--nosystem_rc
--noworkspace_rc
--nohome_rc

And supports finding bazelrc files in all the documented locations
as-per the docs at:
https://docs.bazel.build/versions/2.0.0/guide.html#bazelrc

It can handle command inheritance. So if you specify a "test" command
it knows to look at test, build, and common configs.

It also can deal with import and try-import commands, and can chase
those down to any nesting level.
@philwo
philwo approved these changes Feb 6, 2020
@bazel-io bazel-io closed this in 2b708fb Feb 6, 2020
@elklein elklein deleted the elklein:BashConfigCompletion branch Feb 6, 2020
@philwo

This comment has been minimized.

Copy link
Member

philwo commented Feb 7, 2020

@elklein Thank you for the contribution! This is a very nice feature to have. 😃

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

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.