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(preset): Add No Empty Icons preset #4518

Merged
merged 6 commits into from
Nov 6, 2022

Conversation

Kissaki
Copy link
Contributor

@Kissaki Kissaki commented Oct 22, 2022

When toolset files are identified by the Starship module, the default format displays the toolset icon, and additional information.

When the toolset executables are not available, the additional information (like version number) is missing. Only the toolset icon is displayed.

The No Empty Icons preset changes the format configuration to not show the toolset icons if the variables are empty - presumably because the toolset is not installed or found.

Closes #3070
Related #3248

docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
@Kissaki Kissaki requested a review from davidkna October 31, 2022 16:13
Copy link
Member

@davidkna davidkna left a comment

Choose a reason for hiding this comment

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

As I mentioned in a comment, please remove the repository modules. When creating suggestions, I also noticed a few modules for shell environments which should also be removed.

docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
docs/.vuepress/public/presets/toml/no-empty-icons.toml Outdated Show resolved Hide resolved
Kissaki and others added 6 commits November 1, 2022 13:27
When toolset files are identified by the Starship module, the default format displays the toolset icon, and additional information.

When the toolset executables are not available, the additional information (like version number) is missing. Only the toolset icon is displayed.

The No Empty Icons preset changes the format configuration to not show the toolset icons if the variables are empty - presumably because the toolset is not installed or found.

Closes starship#3070
Related starship#3248
Co-authored-by: David Knaack <davidkna@users.noreply.github.com>
Co-authored-by: David Knaack <davidkna@users.noreply.github.com>
Co-authored-by: David Knaack <davidkna@users.noreply.github.com>
Co-authored-by: David Knaack <davidkna@users.noreply.github.com>
@andytom andytom merged commit 1a3d51f into starship:master Nov 6, 2022
@andytom
Copy link
Member

andytom commented Nov 6, 2022

Thank you for your contribution @Kissaki and thank you for reviewing @davidkna.

Indyandie pushed a commit to Indyandie/starship that referenced this pull request Jul 26, 2023
* feat(preset): Add No Empty Icons preset

When toolset files are identified by the Starship module, the default format displays the toolset icon, and additional information.

When the toolset executables are not available, the additional information (like version number) is missing. Only the toolset icon is displayed.

The No Empty Icons preset changes the format configuration to not show the toolset icons if the variables are empty - presumably because the toolset is not installed or found.

Closes starship#3070
Related starship#3248

* Remove non-version-related modules from preset

Co-authored-by: David Knaack <davidkna@users.noreply.github.com>

* Add new module opa to preset

Co-authored-by: David Knaack <davidkna@users.noreply.github.com>

* Drop unnecessary inner conditional from format

Co-authored-by: David Knaack <davidkna@users.noreply.github.com>

* Remove commit_hash_length setting from preset

* Remove undesired modules

Co-authored-by: David Knaack <davidkna@users.noreply.github.com>

Co-authored-by: David Knaack <davidkna@users.noreply.github.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.

Check for existence of relevant binaries in $PATH
4 participants