Skip to content

Conversation

@tleonhardt
Copy link
Member

@tleonhardt tleonhardt commented Jul 14, 2020

Fix a dependency version requirement which was too restrictive and required bleeding edge for no good reason.

The 1.2.0 release was put out with a dependency on version 1.7.0 or newer of importlib-metadata when running on versions of Python earlier than 3.8. However, that version was only released on June 26, 2020 which makes it extremely bleeding edge: https://pypi.org/project/importlib-metadata/#history

This PR relaxes the dependency to >= 1.6.0 which was released on March 27, 2020 so it is still rather new, but not as painfully bleeding-edge. The point of this is to ease compatibility issues with recent Linux distros and such.

@tleonhardt tleonhardt added the bug label Jul 14, 2020
@tleonhardt tleonhardt added this to the 1.2.1 milestone Jul 14, 2020
@tleonhardt tleonhardt requested a review from kmvanbrunt July 14, 2020 03:08
@tleonhardt tleonhardt requested a review from kotfu as a code owner July 14, 2020 03:08
@tleonhardt tleonhardt self-assigned this Jul 14, 2020
@codecov
Copy link

codecov bot commented Jul 14, 2020

Codecov Report

Merging #956 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #956   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files          19       19           
  Lines        4188     4188           
=======================================
  Hits         4090     4090           
  Misses         98       98           

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 eddbbef...67af98e. Read the comment docs.

@tleonhardt tleonhardt merged commit 683d049 into master Jul 14, 2020
@tleonhardt tleonhardt deleted the importlib-metadata branch July 14, 2020 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants