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

Metadata 42 #45

Merged
merged 3 commits into from
Jan 8, 2020
Merged

Metadata 42 #45

merged 3 commits into from
Jan 8, 2020

Conversation

ewjoachim
Copy link
Contributor

@ewjoachim ewjoachim commented Dec 14, 2019

Fixes #42

Successful PR Checklist:

  • Tests
  • Documentation (optionally: run spell checking)
  • Had a good time contributing? (feel free to give some feedback)

@codecov
Copy link

codecov bot commented Dec 14, 2019

Codecov Report

Merging #45 into master will increase coverage by 0.57%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   56.56%   57.14%   +0.57%     
==========================================
  Files          13       14       +1     
  Lines         594      602       +8     
  Branches       81       81              
==========================================
+ Hits          336      344       +8     
  Misses        251      251              
  Partials        7        7
Flag Coverage Δ
#integration 12.45% <86.66%> (+1.17%) ⬆️
#unit 55.98% <100%> (+0.59%) ⬆️
Impacted Files Coverage Δ
septentrion/metadata.py 100% <100%> (ø)
septentrion/__init__.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99514dd...f8a955b. Read the comment docs.

@mgu
Copy link
Contributor

mgu commented Dec 16, 2019

tests/unit/test_init.py still uses _extract_version

@ewjoachim
Copy link
Contributor Author

@mgu rewrote unit test. We could also have an integration test. (but meh)

@ewjoachim
Copy link
Contributor Author

@mgu ?

Copy link
Contributor

@mgu mgu left a comment

Choose a reason for hiding this comment

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

LGTM thx 🎉

@ewjoachim ewjoachim merged commit 976bc01 into master Jan 8, 2020
@ewjoachim ewjoachim deleted the metadata-42 branch January 8, 2020 10:00
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.

Use importlib_metadata rather than pkg_resources
2 participants