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.

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

Rewrite argument handling #2

Closed
CoolandonRS opened this issue Nov 20, 2023 · 3 comments · Fixed by #4 or #5
Closed

Rewrite argument handling #2

CoolandonRS opened this issue Nov 20, 2023 · 3 comments · Fixed by #4 or #5
Assignees
@CoolandonRS

Comments

@CoolandonRS
Copy link
Owner

It... kinda sucks.
No --help since no dual-tack flags, and no --arg val since space delimination isn't implemented.

The arg-rewrite and arg-rewrite-2 are both for this, with varying degrees of success, where I sketched out a good amount of code and then reflected on whether or not I liked it. There's probably going to be an arg-rewrite-3, if not more.
@CoolandonRS CoolandonRS mentioned this issue Nov 20, 2023
Closed
11 tasks
@CoolandonRS CoolandonRS self-assigned this Nov 23, 2023
@CoolandonRS
Copy link
Owner Author

Deleted arg-rewrite, have decided to go with arg-rewrite-2

@CoolandonRS
Copy link
Owner Author

CoolandonRS commented Nov 23, 2023 •
edited
Loading

Checklist for my convenience:

  • Theoretically Functional
  • Unit Tests
  • Actually Functional
  • Things that should maybe also be done?
    • ArgContracts for stuff like required or incompatible args
    • AliasedArgs? (Probably with a GetAliased method instead of Get for convinence to me)
      • ArgContracts compatibility? (IE: all aliased members are considered incompatible with eachother)
    • Reuseable args?

@CoolandonRS CoolandonRS linked a pull request Jan 23, 2024 that will close this issue
Arg rewrite 2 #3
Merged
@CoolandonRS CoolandonRS reopened this Jan 23, 2024
@CoolandonRS
Copy link
Owner Author

Oops merged it to arg-rewrite-2 and then deleted the branch

@CoolandonRS CoolandonRS removed a link to a pull request Jan 23, 2024
Arg rewrite 2 #3
Merged
@CoolandonRS CoolandonRS linked a pull request Jan 23, 2024 that will close this issue
Finish unit tests #4
Merged
@CoolandonRS CoolandonRS mentioned this issue Jan 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CoolandonRS CoolandonRS

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Finish unit tests Salamander01/consolelib
Arg Rewrite CoolandonRS/consolelib
1 participant
@CoolandonRS