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.

Sign up for GitHub

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

Jump to bottom

Refer to the next version with command #165

Merged
merged 2 commits into from
Dec 31, 2023
Merged

Refer to the next version with command #165

merged 2 commits into from
Dec 31, 2023

Conversation

k2tzumi
Copy link
Contributor

@k2tzumi k2tzumi commented Dec 29, 2023

refs #164

@Songmu
Copy link
Owner

Songmu commented Dec 31, 2023

This is a nice feature. I thought this was necessary.
However, I think that commander.Cmd is a generic function for executing commands, and passing version information to it is not a good separation of duties. I have modified it a bit on my side and will merge them.

@Songmu Songmu merged commit 85c77ac into Songmu:main Dec 31, 2023
3 checks passed
@temporary-token-issuer-for-tagpr temporary-token-issuer-for-tagpr bot mentioned this pull request Dec 31, 2023
Merged
@Songmu Songmu added the minor label Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
minor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@k2tzumi @Songmu