Skip to content

feat(ZVM): Add ZVM segment and docs#5990

Open
ggottemo wants to merge 3 commits intoJanDeDobbeleer:mainfrom
ggottemo:main
Open

feat(ZVM): Add ZVM segment and docs#5990
ggottemo wants to merge 3 commits intoJanDeDobbeleer:mainfrom
ggottemo:main

Conversation

@ggottemo
Copy link

Prerequisites

  • [ x ] I have read and understood the contributing guide.
  • [ x ] The commit message follows the conventional commits guidelines.
  • [ x ] Tests for the changes have been added (for bug fixes / features).
  • [ x ] Docs have been added/updated (for bug fixes / features).

Description

Added a segment for ZVM (Zig Version Manager), that displays the version currently 'in use', similar to NVM (Node Version Manager) in the JavaScript ecosystem.

Changed behavior to turn off colors for zvm when finding the version
@ggottemo
Copy link
Author

Apologies if you already get a notification, wasn't sure if I needed to click 'Re-request review' again or not

valid bool
}

func (c *colorCommand) detectColorState() colorState {
Copy link
Owner

Choose a reason for hiding this comment

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

why is this necessary? That switch should not reset the entire command output for following commands, right? If so, this is terrible design on their part.

@JanDeDobbeleer JanDeDobbeleer force-pushed the main branch 3 times, most recently from eb42120 to 3bf3592 Compare June 25, 2025 09:01
@JanDeDobbeleer JanDeDobbeleer force-pushed the main branch 2 times, most recently from c05927e to ef0ff8b Compare September 11, 2025 09:23
@JanDeDobbeleer
Copy link
Owner

@zaaine I asked a question, can you look at that? Without this answer I can't merge this ;-)

@JanDeDobbeleer JanDeDobbeleer force-pushed the main branch 9 times, most recently from 0f43b02 to fa38ae3 Compare November 7, 2025 15: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.

4 participants