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 simple bash completion #506

Merged
merged 3 commits into from Jul 10, 2018

Conversation

Projects
None yet
3 participants
@scoopex
Contributor

scoopex commented Jul 10, 2018

No description provided.

Marc Schoechlin added some commits Jul 10, 2018

Marc Schoechlin
bash completion
- add simple bash completion
- add documentation
Marc Schoechlin
bash completion
- add simple bash completion
- add documentation
### Bash autocompletion for bolt
### include this file in your .bash_profile or .bashrc to use completion
### "source bolt_bash_completion.sh"
<<<<<<< HEAD

This comment has been minimized.

@lucywyman

lucywyman Jul 10, 2018

Member

It looks like you've still got some git diffs here.

This comment has been minimized.

@scoopex

scoopex Jul 10, 2018

Contributor

i'm sorry :-) Fixed it :-)

This comment has been minimized.

@lucywyman

lucywyman Jul 10, 2018

Member

You might need to make sure you push to your branch - I'm not seeing it fixed

Marc Schoechlin
@lucywyman

This comment has been minimized.

Member

lucywyman commented Jul 10, 2018

@scoopex Can you squash the 2 commits into 1 commit? And perhaps add more details to your commit message about what you're adding, how to use it, and why? Doesn't need to be long, just a line or 2 :)

git rebase -i HEAD~2

Then change the "pick" on the second line to "f". Let us know if you need help or want us to do it for ya!

@MikaelSmith MikaelSmith merged commit 770e13e into puppetlabs:master Jul 10, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment