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

Cli ux improvements docs #1328

Merged
merged 6 commits into from
Jan 9, 2025
Merged

Conversation

elcritch
Copy link
Contributor

@elcritch elcritch commented Jan 9, 2025

Ooops, forgot to update some of the help docs for my last PR. The new commands aren't helpful if folks can't find them!

Also decided to change nimble publish --tags to nimble publishTags. It helps split out options for the commands and their options, while still grouping similar actions for discoverability.

@Araq Araq merged commit cff3719 into nim-lang:master Jan 9, 2025
11 checks passed
@arnetheduck
Copy link

nimble publishTags feels wrong / too much though. nimble tag would make more sense, where it would read the nimble file and put a corresponding git tag and do nothing else (ie not push the tag to any remotes). in particular, changing past tags is a big no-no and the commit that changes the nimble file is not always the desired release version.

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