Skip to content

Cleanup output from --version#16

Merged
adityaramani merged 1 commit intomainfrom
users/a_ramani/version-output
Jun 5, 2025
Merged

Cleanup output from --version#16
adityaramani merged 1 commit intomainfrom
users/a_ramani/version-output

Conversation

@adityaramani
Copy link
Contributor

@adityaramani adityaramani commented Jun 5, 2025

The output now looks like the following

When build against a tagged HEAD in DEBUG mode

container --version
container CLI version 0.0.2 (build: debug, commit: ed4a4cb)

When build against a tagged HEAD in RELEASE mode

container --version
container CLI version 0.0.2 (build: release, commit: ed4a4cb)

When build against a non tagged HEAD in DEBUG mode

container --version
container CLI version 0.0.1-dev-5-g87a989d (build: debug, commit: 87a989d)

When build against a non tagged HEAD in DEBUG mode when the CURRENT_SDK environment variable is set

container --version
container CLI version 0.0.1-dev-5-g87a989d (build: debug, commit: 87a989d, sdk: MacOS 15)

@egernst
Copy link
Contributor

egernst commented Jun 5, 2025

Thanks Aditya. Is there utility in still having the containerization version shown?

@adityaramani adityaramani merged commit a756184 into main Jun 5, 2025
2 checks passed
@adityaramani adityaramani deleted the users/a_ramani/version-output branch June 5, 2025 16:10
katiewasnothere pushed a commit that referenced this pull request Jun 5, 2025
@adityaramani @katiewasnothere
Cleanup output from --version (#16)
e5aceaa 
The output now looks like the following

#### When build against a tagged HEAD in `RELEASE` mode
```
container --version
container CLI version 0.0.2 (build: release, commit: ed4a4cb)
```
owenarthur pushed a commit to owenarthur/container that referenced this pull request Jun 10, 2025
@adityaramani @katiewasnothere
Cleanup output from --version (apple#16)
e1bc3ca 
The output now looks like the following

#### When build against a tagged HEAD in `RELEASE` mode
```
container --version
container CLI version 0.0.2 (build: release, commit: ed4a4cb)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jglogan jglogan jglogan approved these changes

@egernst egernst Awaiting requested review from egernst

@crosbymichael crosbymichael Awaiting requested review from crosbymichael

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@adityaramani @egernst @jglogan @katiewasnothere