Skip to content

Fix for Issue #32 arguments not working#41

Open
jkcko wants to merge 4 commits into
docopt:masterfrom
jkcko:master
Open

Fix for Issue #32 arguments not working#41
jkcko wants to merge 4 commits into
docopt:masterfrom
jkcko:master

Conversation

@jkcko

@jkcko jkcko commented Aug 24, 2018

Copy link
Copy Markdown

I would like my changes to support positional arguments (Issue #32) to be considered for merging.

The naval fate example now works with options before or after the positional arguments.

@SamuelMarks

Copy link
Copy Markdown
Contributor

@jkcko Looks like this is unmaintained… I've added support for C89 and a few minor linting improvements: https://github.com/offscale/docopt.c/tree/c

Would love to merge your changes 😄

@jkcko

jkcko commented May 18, 2020

Copy link
Copy Markdown
Author

@jkcko Looks like this is unmaintained… I've added support for C89 and a few minor linting improvements: https://github.com/offscale/docopt.c/tree/c

Would love to merge your changes smile

@SamuelMarks Looks like that last merge was back on Sep 26, 2016. Looks like the maintainer has disappeared and that's unfortunate. Perhaps there just wasn't enough interest in the C version of docopt. Did you merge my changes to your own fork and try them yourself?

@SamuelMarks

Copy link
Copy Markdown
Contributor

Unfortunately I only just found your PR… happy to accept new PRs to my fork

There are a few new features I've added, and a bunch more I have planned. The C89 support was a little finicky, but now that is valid also.

Yeah, it's annoying that @kblomqvist @ffunenga and @keleshev disappeared :\

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