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.

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

Fix for Issue #32 arguments not working #41

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jkcko
Copy link

@jkcko jkcko commented Aug 24, 2018

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
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
Copy link
Author

jkcko commented May 18, 2020

@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
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