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

feat: shell completions #195

Merged
merged 10 commits into from
Sep 29, 2022
Merged

feat: shell completions #195

merged 10 commits into from
Sep 29, 2022

Conversation

istudyatuni
Copy link
Contributor

@istudyatuni istudyatuni commented Sep 12, 2022

Close #193

Depends on #194

Not sure where is better place to write docs about installing completions. Also that this way for generating is good

src/commands/mod.rs Outdated Show resolved Hide resolved
.github/workflows/release.yaml Outdated Show resolved Hide resolved
@aawsome
Copy link
Member

aawsome commented Sep 27, 2022

@istudyatuni Are you still working on this PR? I would love to add this feature to the next release which I'm planning to release within the next days.
Also, please give me a hint if you need any kind of support!

@istudyatuni
Copy link
Contributor Author

Yeah, sorry, studying now a lot. Will return to this in 1-3 days (I hope). Also question - maybe do not hide completions behind feature, and provide this by default? Like in poetry

@aawsome
Copy link
Member

aawsome commented Sep 29, 2022

No worries and no need to hurry. I'm just trying to get things closed which can be closed 😉

Also question - maybe do not hide completions behind feature, and provide this by default? Like in poetry

Yes, this is for sure an option.

@istudyatuni istudyatuni marked this pull request as ready for review September 29, 2022 15:10
@istudyatuni
Copy link
Contributor Author

Done

Copy link
Member

@aawsome aawsome left a comment

Choose a reason for hiding this comment

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

Thanks, looks pretty good to me.
Can you squash your commits into 1 commit?

.github/workflows/check-and-lint.yaml Show resolved Hide resolved
@istudyatuni
Copy link
Contributor Author

About squashing - maybe make a squash merge?

Copy link
Member

@aawsome aawsome left a comment

Choose a reason for hiding this comment

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

LGTM

.github/workflows/check-and-lint.yaml Show resolved Hide resolved
@aawsome aawsome merged commit 55e0b81 into rustic-rs:main Sep 29, 2022
@istudyatuni istudyatuni deleted the feat/shell-completions branch September 30, 2022 05:30
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.

Make shell completions
2 participants