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

Improve help prompt value names #153

Merged
merged 1 commit into from
Nov 5, 2022

Conversation

tranzystorekk
Copy link
Contributor

Standards checklist:

  • The PR title is descriptive.
  • The code compiles (cargo build)
  • The code passes rustfmt (cargo fmt)
  • The code passes clippy (cargo clippy)
  • The code passes tests (cargo test)
  • Optional: I have tested the code myself
    • I also tested that Topgrade skips the step where needed

If you developed a feature or a bug fix for someone else and you do not have the
means to test it, please tag this person here.

@pan93412
Copy link
Member

pan93412 commented Nov 5, 2022

Excellent PR! Visual Changes:

截圖 2022-11-05 下午8 04 27

截圖 2022-11-05 下午8 04 39

@tranzystorekk
Copy link
Contributor Author

the custom-command logic could probably use some improvement, as it seems to always run all the items from custom commands instead of just the selected ones

@s34m
Copy link
Member

s34m commented Nov 5, 2022

Could you change the branch to the bug-fixes branch.

@tranzystorekk tranzystorekk changed the base branch from master to bug-fixes November 5, 2022 15:31
@tranzystorekk
Copy link
Contributor Author

Done

@s34m s34m merged commit c6f180d into topgrade-rs:bug-fixes Nov 5, 2022
@tranzystorekk tranzystorekk deleted the clap-value-names branch November 5, 2022 16:16
s34m added a commit that referenced this pull request Nov 6, 2022
* Closes #150 please disable distrobox by default (#151)

* Check if distrobox exists before running step

* Improve help prompt value names (#153)

* 159 self update error message with standalone versions (#161)

* Rename back to topgrade

* Bugfix Version bump

* Changes reference to topgrade-rs in self-update

* Fixes distrobox errors (#160)

* Rename back to topgrade

* Bugfix Version bump

* Check if distrobox exists before running step

* Fixed sitrobox and version bump

* Version bump to 10.1.2

Co-authored-by: Marcin Puc <tranzystorek.io@protonmail.com>
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.

3 participants