Skip to content

Remove release subcommands #3275

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
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Remove release subcommands #3275

wants to merge 2 commits into from

Conversation

@josephperrott
Copy link
Member

@josephperrott josephperrott commented Dec 1, 2025

  • Delete the deprecated and unused set-dist-tag command
  • Remove all of the ng-dev release subcommands from the listings
  • Move ng-dev release publish to be called as ng-dev release instead.

@josephperrott
fix(ng-dev): complete the removal of ng-dev release set-dist-tag as i…
6262e87 
…ts no longer used
@josephperrott
refactor(ng-dev): remove subcommands from ng-dev release
e3f889e 
Remove all of the subcommands from the listing of ng-dev commands and leave only ng-dev release command as this is the only one
needed by end users.  The other subcommands still exist and are able to be called, but do not appear in the help menus.
@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Dec 2, 2025
@devversion
Copy link
Member

devversion commented Dec 3, 2025

I wasn't follow much of the work here, but in the past, the set-dist-tag command was super useful when the release tool failed randomly mid-publishing

@josephperrott
Copy link
Member Author

josephperrott commented Dec 4, 2025

set-dist-tag is a noop now, it was replaced by npm-dist-tag. Ironically, it was you that made the change and marked for us to remove set-dist-tag after it was no longer needed by any LTS versions

@devversion
Copy link
Member

devversion commented Dec 4, 2025

Oh haha, I didn't realize we have two at this point. Thanks for the clean-up!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devversion devversion devversion approved these changes

@alan-agius4 alan-agius4 alan-agius4 approved these changes

Assignees

No one assigned

Labels

action: merge The PR is ready for merge by the caretaker

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@josephperrott @devversion @alan-agius4