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

Add __main__.py #503

Merged
merged 3 commits into from
Sep 17, 2021
Merged

Add __main__.py #503

merged 3 commits into from
Sep 17, 2021

Conversation

pinkwah
Copy link
Contributor

@pinkwah pinkwah commented Sep 14, 2021

This will allow users to do python -m webviz_config [...] in addition to webviz [...].


Contributor checklist

  • 🎉 This PR closes #ISSUE_NUMBER.
  • 📜 I have broken down my PR into the following tasks:
    • Task 1
    • Task 2
  • 🤖 I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR.
  • 📖 I have considered adding a new entry in CHANGELOG.md, and added it if should be communicated there.

This will allow users to do 'python -m webviz_config [...]' in addition
to 'webviz [...]'
@anders-kiaer anders-kiaer changed the title Add __main__.py Add __main__.py Sep 14, 2021
@anders-kiaer
Copy link
Collaborator

anders-kiaer commented Sep 14, 2021

Thanks, LGTM @dotfloat 👍 Could you add a short changelog entry in CHANGELOG.md under [UNRELEASED], and then we merge? E.g. something like:

### Added
- [#503](https://github.com/equinor/webviz-config/pull/503) - Added `__main__.py`. This will allow users to do `python -m webviz_config [...]` in addition to `webviz [...]`.

Copy link
Collaborator

@anders-kiaer anders-kiaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@anders-kiaer anders-kiaer merged commit 21a33e4 into equinor:master Sep 17, 2021
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