Skip to content

fix(cli): correct positional argument parsing#3

Merged
Conaclos merged 1 commit into
bare-ts:mainfrom
merlindiavova:fix-positional-arguments
Oct 12, 2025
Merged

fix(cli): correct positional argument parsing#3
Conaclos merged 1 commit into
bare-ts:mainfrom
merlindiavova:fix-positional-arguments

Conversation

@merlindiavova

@merlindiavova merlindiavova commented Sep 23, 2025

Copy link
Copy Markdown
Contributor
  • Fix handling of 'compile' command in positional arguments
  • Ensure schema file is properly identified when using compile command
  • Prevent incorrect assignment of output file as schema input
  • Maintain backward compatibility with existing command usage

Addresses: #2

- Fix handling of 'compile' command in positional arguments
- Ensure schema file is properly identified when using compile command
- Prevent incorrect assignment of output file as schema input
- Maintain backward compatibility with existing command usage
@Conaclos Conaclos force-pushed the main branch 3 times, most recently from bdfa5de to 3ef9d2f Compare October 12, 2025 21:09

@Conaclos Conaclos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @merlindiavova

Thanks for sending a fix :)
My repository watcher was misconfigured. Thus. I never received a notification about the PR you submitted.

Your fix looks good to me!

@Conaclos Conaclos force-pushed the main branch 2 times, most recently from fdfc763 to defc53d Compare October 12, 2025 22:46
@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 17943924173

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.016%

Totals Coverage Status
Change from base Build 18450554429: 0.0%
Covered Lines: 3416
Relevant Lines: 3601

💛 - Coveralls

@Conaclos Conaclos merged commit 78389ac into bare-ts:main Oct 12, 2025
1 of 2 checks passed
Conaclos pushed a commit that referenced this pull request Oct 12, 2025
- Fix handling of 'compile' command in positional arguments
- Ensure schema file is properly identified when using compile command
- Prevent incorrect assignment of output file as schema input
- Maintain backward compatibility with existing command usage

Co-authored-by: Merlin Diavova <merlin@amespalsh.co.uk>
Conaclos pushed a commit that referenced this pull request Oct 12, 2025
- Fix handling of 'compile' command in positional arguments
- Ensure schema file is properly identified when using compile command
- Prevent incorrect assignment of output file as schema input
- Maintain backward compatibility with existing command usage

Co-authored-by: Merlin Diavova <merlin@amespalsh.co.uk>
Conaclos pushed a commit that referenced this pull request Oct 12, 2025
- Fix handling of 'compile' command in positional arguments
- Ensure schema file is properly identified when using compile command
- Prevent incorrect assignment of output file as schema input
- Maintain backward compatibility with existing command usage

Co-authored-by: Merlin Diavova <merlin@amespalsh.co.uk>
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