-
Notifications
You must be signed in to change notification settings - Fork 3
feat(*): Allow multiple files to be used for parsing #9
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
base: master
Are you sure you want to change the base?
feat(*): Allow multiple files to be used for parsing #9
Conversation
| - Run `pipenv run python serialize_data.py` to generate JSON file named `endpoints_data.json`. Optionally, you can specify the path of the proposed endpoints docs using `--file-path` option like: | ||
| - Run `pipenv run python serialize_data.py` to generate JSON file named `endpoints_data.json`. Optionally, you can specify the path of the proposed endpoints docs using `--file-path` or `-f` option like: | ||
|
|
||
| ```pipenv run python serialize_data.py --file-path /pat/to/endpoints_data.json``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess, the endpoints_data.json was wrong in this command earlier? It needed a .md file instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. The last commit does that.
6fdaf21 to
6bc7720
Compare
README.md
Outdated
|
|
||
| You can also provide multiple files as | ||
|
|
||
| ```pipenv run python serialize_data.py -f /path/to/proposed_endpoints.md -f path/to/another_proposed_endpoints.md``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The docs does not have any mention for running like this:
pipenv run python serialize_data.py *.md
You've this format mentioned in the docstring though 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated in the latest commit.
|
|
||
| If files resides at different paths, then you may provide them as positional argument to it. For e.g | ||
|
|
||
| ```pipenv run python serialize_data.py /path/to/some_file.md /path/to/another_file.md``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Show in example that you meant another path ;)
This would allow multiple files to be used to generate the stub endpoints
json.