Skip to content

Fix --version being used twice + use parseArgs #5

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
Merged
RomainCscn merged 2 commits into from
Feb 6, 2026
Merged

Fix --version being used twice + use parseArgs #5

RomainCscn merged 2 commits into from
Feb 6, 2026

Conversation

@RomainCscn
Copy link
Collaborator

@RomainCscn RomainCscn commented Feb 6, 2026 •
edited
Loading

  • Rename --version flag to --release-version to avoid conflict with -v/--version (show version)
  • Extract parseCLIArgs into src/args.ts with test coverage
  • Wrap arg parsing in try/catch for clean error messages on invalid flags

Fixes LIN-58727
Fixes LIN-58887

@RomainCscn RomainCscn mentioned this pull request Feb 6, 2026
Merged
@RomainCscn RomainCscn merged commit db481c7 into main Feb 6, 2026
2 checks passed
@RomainCscn RomainCscn deleted the romain/fix-release-version branch February 6, 2026 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mwolting mwolting mwolting approved these changes

@axelniklasson axelniklasson Awaiting requested review from axelniklasson

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@RomainCscn @mwolting