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

Expand metaflow configure with export and import. #53

Merged
merged 3 commits into from
Dec 16, 2019

Conversation

crk-codaio
Copy link
Contributor

  • Introduce metaflow configure export [OUTPUT_FILENAME] to export an
    existing configuration to a file.
  • In similar vein, also add metaflow configure import [INPUT_FILENAME]
    to import the configuration (perhaps outputted by export)
  • Pretty print the JSON configuration everywhere so its more
    human-readable.
  • Improve the handling of missing profile(s) consistently throughout
    show/reset/export/import.

* Introduce `metaflow configure export [OUTPUT_FILENAME]` to export an
existing configuration to a file.
* In similar vein, also add `metaflow configure import [INPUT_FILENAME]`
to import the configuration (perhaps outputted by export)
* Pretty print the JSON configuration everywhere so its more
human-readable.
* Improve the handling of missing profile(s) consistently throughout
show/reset/export/import.
metaflow/main_cli.py Outdated Show resolved Hide resolved
metaflow/main_cli.py Outdated Show resolved Hide resolved
metaflow/main_cli.py Outdated Show resolved Hide resolved
metaflow/main_cli.py Show resolved Hide resolved
* Fix exporting absent (default) profile.
@crk-codaio
Copy link
Contributor Author

@savingoyal Please merge this along with the rest whenever you're ready for the release. I'll leave it as-is (hopefully no merge conflicts :))

@savingoyal savingoyal merged commit 5e52934 into master Dec 16, 2019
@savingoyal savingoyal deleted the configure-export-import branch December 19, 2019 01:59
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