Skip to content

Comments

Implement Basic Procedures for Argument Parsing#12

Merged
ufo5260987423 merged 2 commits intoMoganLab:mainfrom
rennsax:main
Jun 2, 2025
Merged

Implement Basic Procedures for Argument Parsing#12
ufo5260987423 merged 2 commits intoMoganLab:mainfrom
rennsax:main

Conversation

@rennsax
Copy link

@rennsax rennsax commented Jun 2, 2025

Implement prototype for #11.

A new record type scheme-lsp-args is defined. Currently, only the oprands field is used, to keep the command line interface consistent.

Further plan: we may adjust the user experience of the CLI. For example, specify multi-thread with options, not oprands.

@ufo5260987423 ufo5260987423 merged commit 5777207 into MoganLab:main Jun 2, 2025
1 check passed
@ufo5260987423
Copy link
Collaborator

I accidently merged. But this pr is actually not correct.
The question is that:

  1. We don't need scheme-lsp-args, not only there's a lsp named scheme-lsp-server, but also this record type won't be used twice in this program;
  2. you should keep the help information and usage identical to origin, changing this information is actually a big step with version increasing.

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.

2 participants