-
Notifications
You must be signed in to change notification settings - Fork 70
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
Configure with meta/config. #243
Conversation
Add support for Python 3.9.
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.
The tests are crashing because coverage can't find a plugin:
Run coverage run -p -m unittest discover -s src
Traceback (most recent call last):
File "/opt/hostedtoolcache/Python/3.6.13/x64/bin/coverage", line 8, in <module>
sys.exit(main())
File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/coverage/cmdline.py", line 871, in main
status = CoverageScript().command_line(argv)
File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/coverage/cmdline.py", line 588, in command_line
return self.do_run(options, args)
File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/coverage/cmdline.py", line 743, in do_run
self.coverage.start()
File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/coverage/control.py", line 532, in start
self._init()
File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/coverage/control.py", line 275, in _init
self._plugins = Plugins.load_plugins(self.config.plugins, self.config, self._debug)
File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/coverage/plugin_support.py", line 41, in load_plugins
__import__(module)
ModuleNotFoundError: No module named 'coverage_python_version'
Now CI fails at pushing data to coveralls.io because This step is done via the GitHub action https://github.com/AndreMiras/coveralls-python-action and I do not see a chance there to install It seems that we either have to remove |
I wonder that nobody else complained about this problem in https://github.com/AndreMiras/coveralls-python-action/issues. Do people rarely use coverage plugins? For my personal projects I've been uploading to coveralls.io by using the usual One issue with that approach is that you need a recent enough version of https://pypi.org/p/coveralls to support GITHUB_TOKEN, and that version no longer supports Python 2. |
That's one of the major reasons I've been using the docker version everywhere.
I think that's right too. |
I've filed an issue: AndreMiras/coveralls-python-action#17 |
Let's see if there will be a reaction to AndreMiras/coveralls-python-action#17. Otherwise I'd drop the usage of |
Oh, we're not actually using the coverage-python-version plugin here? Then let's by all means drop it. |
Dropped |
There is no need for zope.testrunner. Drop leftovers of `coverage_python_version`.
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.
The tox.ini setup for -pure environments doesn't work; the rest LGTM.
pragma: no cover | ||
class I[A-Z]\w+\((Interface|I[A-Z].*)\): |
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.
How much coverage are we losing by dropping this from exclude_lines?
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.
I hopefully fixed all the places where this line ignored methods by changing pass
to a docstring.
There is still https://coveralls.io/builds/39516428/source?filename=src%2Fzope%2Finterface%2Fcommon%2Fcollections.py#L221 which I do not understand as I thought we have coverage for all Python versions. (There are some other cases which look the same.)
@mgedmin You requested changes to this PR. I think I fixed them. Are you okay with them? |
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
Thank you for reviewing this PR. 😃 |
Add support for Python 3.9.
See zopefoundation/meta#94 for the needed changes in meta/config.