Hal Blackburn
Hal Blackburn
Thanks @NickCrews, glad to hear & no problem at all, thanks for the review, I should follow up later on today.
Just pushed a change to import `Sequence` from `typing` instead of `collections.abc` as that won't work on Python 3.7.
Related issue in the docopt repo: https://github.com/docopt/docopt/issues/259
Oh ok, no probs. No rush.
CI should pass now, I fixed the 3.7/3.8 compatibility issue. I should have covered everything, but need to discuss the `-not-an-option` thing and behaviour of the usage_pattern regex.
Oh, also, fixing the `testcases.docopt` tests revealed an option description parsing bug which turned out to be a pre-existing issue, see commit [45f2a19](https://github.com/jazzband/docopt-ng/pull/36/commits/45f2a196069c5a151b3f970f28c7b0f96ab8e0e3) here.
The simple fixes are not-yet-squashed fixup commits, I can rebase/squash to cleanup the history before merging as and when we're ready.
@NickCrews Great, thanks for all your help reviewing this, it's really helpful. And sorry to make things trickier with that fix. That's the "fix: handle options with descriptions on new...
@NickCrews I fixed those two things. Let me know what you'd like to do with the option defaults newline fix. I pushed a separate branch to my repo with the...
@NickCrews I've updated this PR's branch to the squashed & rebased version (same as #38) so you can merge here if you're happy.