Skip to content

Conversation

@gutsytechster
Copy link

This would allow multiple files to be used to generate the stub endpoints json.

- 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```
Copy link
Owner

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?

Copy link
Author

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.

@gutsytechster gutsytechster force-pushed the allow_multiple_files branch from 6fdaf21 to 6bc7720 Compare July 12, 2020 09:05
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```
Copy link
Owner

@CuriousLearner CuriousLearner Jul 18, 2020

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 🤔

Copy link
Author

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```
Copy link
Owner

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 ;)

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