Skip to content

Conversation

dvzrv
Copy link
Contributor

@dvzrv dvzrv commented Sep 26, 2022

Fixes #174
Fixes #154

pyproject.toml:
Update the pytest dependency to >= 7.0.0, which obsoletes the use of
pytest-pythonpath.

setup.cfg:
Change the python_paths pytest configuration option to pythonpath, as
that is what is used with pytest >= 7.0.0."
poetry.lock:
Update poetry lock file after updating pytest version requirement and
removing pytest-pythonpath.
Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks!

pyproject.toml:
Using importlib_metadata it is possible to derive versions of packages.
To make use of it with python < 3.10 the package importlib_metadata has
to be added (conditionally).
poetry.lock:
Update poetry lock after adding importlib_metadata conditionally.
@dvzrv dvzrv force-pushed the fix_pytest_and_importlib branch from 35202a7 to 91feca4 Compare September 26, 2022 10:08
@dvzrv
Copy link
Contributor Author

dvzrv commented Sep 26, 2022

Ouf, that CI is very flaky! :S

@dvzrv dvzrv force-pushed the fix_pytest_and_importlib branch 2 times, most recently from 4f2caf0 to 53e40dd Compare September 26, 2022 10:42
setup.cfg:
Disable the warning for unused ignores in mypy as there seems to be an
issue between versions of mypy about what is considered an "unused
ignore".
coverage_conditional_plugin/__init__.py:
Use importlib_metadata to derive package versions, instead of relying on
pkg_resources (which implicitly dragged in setuptools).
@dvzrv dvzrv force-pushed the fix_pytest_and_importlib branch from 53e40dd to 078baf9 Compare September 26, 2022 10:45
setup.cfg:
Disable WPS440 and WPS433 for plugin as they are requirements for the
conditional import of importlib_metadata.
@codecov
Copy link

codecov bot commented Sep 26, 2022

Codecov Report

Merging #177 (2b586dc) into master (98945af) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #177   +/-   ##
=======================================
  Coverage   90.90%   90.90%           
=======================================
  Files           1        1           
  Lines          11       11           
=======================================
  Hits           10       10           
  Misses          1        1           

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

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks!

@sobolevn sobolevn merged commit 66a1444 into wemake-services:master Sep 26, 2022
archlinux-github pushed a commit to archlinux/repod that referenced this pull request Sep 27, 2022
pyproject.toml:
Update dependencies to their latest versions.
Remove setuptools, as coverage-conditional-plugin no longer implicitly
requires it
(wemake-services/coverage-conditional-plugin#177).
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.

Add setuptools to requirements or use modern version to derive version Remove pytest-pythonpath
2 participants