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 command-line completions for fish shell #1760

Merged
merged 1 commit into from
May 4, 2017
Merged

Conversation

smith
Copy link

@smith smith commented May 4, 2017

Since the command to enable them is different, also make that change in
the output based on the shell used.

tenor-195720707

Copy link
Contributor

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

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

Great addition @smith Once the lint issues are fixed, LGTM

Since the command to enable them is different, also make that change in
the output based on the shell used.

Signed-off-by: Nathan L Smith <smith@chef.io>
@smith
Copy link
Author

smith commented May 4, 2017

Great addition @smith Once the lint issues are fixed, LGTM

Fixed.

Do we have any plans to move from rubocop to chefstyle to make the coding style guides consistent across Ruby-based projects? I didn't catch these linting errors because my editor is set up to use chefstyle.

@adamleff
Copy link
Contributor

adamleff commented May 4, 2017

@smith It has been discussed but is not currently planned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Improves an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants