-
Notifications
You must be signed in to change notification settings - Fork 11
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
Make it possible to configure publisher with a dictionary of settings #35
Conversation
Codecov Report
@@ Coverage Diff @@
## main #35 +/- ##
==========================================
+ Coverage 78.57% 79.50% +0.93%
==========================================
Files 15 15
Lines 1792 1888 +96
==========================================
+ Hits 1408 1501 +93
- Misses 384 387 +3
Continue to review full report at Codecov.
|
…uu/posttroll into feature-publisher-dict_config
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.
LGTM. I'm not really happy with the tests checking how Publisher is called, it feels like we are testing the implementation rather than the behaviour of the class. But I think that making Publisher testable needs much more work than is reasonable within the scope of this PR, so I'm merging this.
Add an option to use publisher configuration from a file. Needs merger of posttroll #35 pytroll/posttroll#35
To make publisher selection easier for the user, this PR adds an utility function that selects the publisher based on the given settings.