Use importlib.metadata rather than pkg_resources for version metadata

[norlandrhagen]

Description of proposed changes

PR for Issue #89

Fixes #89

cc @maxrjones

[maxrjones]

Thanks @norlandrhagen!

What's the motivation for placing the code in a new module rather than __init__.py?

[norlandrhagen]

No strong motivation for separating out version from __init__. Just decided to mirror Pandas and cmip6. Will gladly move it back into __init__.py if you think that's best.

[maxrjones]

No strong motivation for separating out version from __init__. Just decided to mirror Pandas and cmip6. Will gladly move it back into __init__.py if you think that's best.

Thanks for the explanation! I think since this is a small amount of code relative to those examples we could minimize the number of modules and keep it in __init__. After that it'll be great to merge this in!

[codecov-commenter]

Codecov Report

Merging #97 (266edf0) into main (f13b125) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #97   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines          182       182           
  Branches        37        33    -4     
=========================================
  Hits           182       182           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[weiji14]

Not too familiar with pkg_resources/importlib, but looks ok to me!

[maxrjones]

A couple suggestions so that the metadata imports aren't included in xbatcher's list of public objects.

[maxrjones]

Thanks @norlandrhagen!