Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@bamarsha
Copy link
Contributor

@bamarsha bamarsha commented Apr 23, 2020

  • Refactor the argument parsers so they support both single-valued and many-valued arguments (arrays). This was done by primarily supporting many-valued arguments (the more general case), and then calling Single() when the argument is single-valued.
  • Pass the argument-parsing functions directly to the Option<T> constructors instead of creating Argument<T> instances and setting the Option<T>.Argument property. These should be functionally equivalent, it is just recommended to do it this way in System.CommandLine.
  • Various cleanup:
    • Split EntryPointDriver.cs into three files.
    • Rename Result<T> to Validation<T> to avoid confusion with Q# Result.

@bamarsha bamarsha requested a review from bettinaheim April 23, 2020 23:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants