forked from docopt/docopts
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update from original - master #1
Open
jaytoday
wants to merge
231
commits into
jaytoday:master
Choose a base branch
from
docopt:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ed a (perhaps unwarranted) major interface change: all arguments `docopts` accepts are now options, separated from the arguments to be parsed by a `--`. `--help=<msg>` is now how the doc string is given to `docopts`. The optional version message is given similarly: `--version=<msg>`. `docopts`'s own help and version messages can be read by calling it with the respective options but without any arguments.
Because I think `docopts -h docstring : arguments` looks nicer and, amazingly enough, docopt actually supports it (now that I can use `options_first`.)
Did I really never test for this since I changed the invocation syntax?
…t call Guess this is a pretty rare use case since no one complained?
setup.py doesn't work anyway when called outside the package directory
io.StringIO is also available in Python 2.7, but defaults to manipulating unicode strings, where Python 2.7 uses byte string everywhere else. You can see how this can become a problem.
By dropping whitespace sensitivity in the greps. Fixes #4.
quick_example.sh not tested
some typos
add comment README.rsti add submodule bats for unit testing add simple docopts.sh and unit tests add get_docopt.sh for downloading docopt.py modified quick_example.sh just work do nothing
…into shell-api-v2 publish first tests
add calculator_example.sh - just print as quick_example.sh modified quick_example.sh - use a working wrapper to docopts.py modified docopts.sh - wrapper to docopts.py pass all args add cat-n_wrapper_example.sh - use agrs multiple example
add docopt_get_eval_array add docopt_get_values
format markdown was awful on github.
…' into pr-52-double-dash-no-mangle merging initial change in the pr branch
…ts into pr-52-double-dash-no-mangle
added example with `double_dash.sh` fix `language_agnostic_tester.py` corner case compare JSON. fix `legacy_bash/cat-n_wrapper_example.sh` was not working at all.
-G mode now support both mangling `-` and `--` respectively: ``` ARGS__=true ARGS___=true ```
Add explanation on `-G` with double-dash and single-dash
* ci: migrate from Travis CI to GitHub Actions - Travis CI is only _pseudo_-free after the .org -> .com merge - c.f. https://travis-ci.community/t/org-com-migration-unexpectedly-comes-with-a-plan-change-for-oss-what-exactly-is-the-new-deal/10567 - GH Actions are _actually_ free for OSS / public repos - and also offer a decent level of composability that's resulted in a huge surge in reusable community Actions (vs. CircleCI orbs etc haven't had nearly as much adoption) - and I've had a mostly great experience on them so far as well - add a matrix for OSes and different Go versions too - currently only using one Go version (as it was on Travis), but can add more later - currently only testing latest Ubuntu and macOS (as it was on Travis), but can add more versions and OSes later - note that tests fail on Windows right now (due to the shell?) - check for `$RUNNER_OS` instead of `$TRAVIS_OS_NAME` now - c.f. https://docs.github.com/en/actions/learn-github-actions/environment-variables - update the commented out "Debug" step to conditionally run on macOS as well * fix: make get_docopts work on macOS - tests failing on macOS b/c `sha256sum` doesn't exist - only use the third arg in the workaround as it doesn't support flags * run on PRs as well
- per the deleted in-line comments, the reason it was installed as a static binary was because Brew's auto-update is slow - we can disable the auto-update though, so do that instead in order to install via Brew
- now that GH Actions are merged, thoroughly tested, and working: - rename `travis` dir to `ci` dir instead - update `release.md` to remove Travis references and reference GH Actions files instead - and just generic "ci build" instead of "travis build" - CI is working now, so uncomment/unquote the previous numbers - leave `cp_to_travis` and `reverse_ssh_tunnel` as is for now, in case Sylvain wants to adapt them to GH Actions - I'm not sure they'll work as-is on GH Actions (e.g. `travis_base` won't exist), so they may need further modification - (or it may not be possible at all with GH Actions to do reverse SSH) - that is beyond my scope since I'm not using these, don't have a bounce host, and am developing on a Mac locally
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Merging latest changes from the original repository.