Add seq-based overload for Docopt.Parse #2525
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fix #2433 by adding an overload to Docopt.Parse that takes an
IEnumerable<string>
. All existing unit tests continue to pass after this PR.TODO
Feel free to open the PR and ask for help
help/markdown
):Help needed: there's no API documentation for Docopt yet, and as I tried to write it, I realized I don't quite know what to write to explain the main Parse() implementation.
Help needed: there are no unit tests for Docopt, and I don't feel up to writing them at the moment.
This is a one-line PR, so I didn't really look at any other code except the Docopt class, and I didn't see anything in the Docopt class that needed fixing.
help/templates/template.cshtml
, linking to the API-reference is fine.N/A
N/A
dotnet sln Fake.sln add src/app/Fake.*/Fake.*.fsproj
)N/A