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

Run tests on GHA. #37

Merged
merged 5 commits into from
Dec 22, 2020
Merged

Run tests on GHA. #37

merged 5 commits into from
Dec 22, 2020

Conversation

icemac
Copy link
Member

@icemac icemac commented Dec 18, 2020

No description provided.

@icemac icemac requested a review from mgedmin December 18, 2020 07:14
@icemac
Copy link
Member Author

icemac commented Dec 18, 2020

CI seems to be broken because of actions/setup-python#171.

@icemac
Copy link
Member Author

icemac commented Dec 18, 2020

Appveyor seems to need a 3.9 wheel of zope.proxy for windows.

@icemac
Copy link
Member Author

icemac commented Dec 18, 2020

Using an older actions/setup-python to get GHA running again.

icemac pushed a commit to zopefoundation/meta that referenced this pull request Dec 18, 2020
Copy link
Member

@mgedmin mgedmin left a comment

Choose a reason for hiding this comment

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

Mostly LGTM, some slight comments.

.github/workflows/tests.yml Outdated Show resolved Hide resolved
python3.7
commands =
coverage run -m zope.testrunner --test-path=src []
coverage report --fail-under=100
Copy link
Member

Choose a reason for hiding this comment

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

We had 100%, and now we have 99%? What happened?

Copy link
Member

Choose a reason for hiding this comment

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

It's mostly branch coverage (which we had disabled before), but also now line 109 in src/zope/sendmail/maildir.py is no longer covered.

This appears to be a side effect of switching from basepython = python3.7 to basepython = python3 (which is 3.8 on my Ubuntu 20.10)!

Anyway, this is fine, we can climb back to 100% later.

@mgedmin
Copy link
Member

mgedmin commented Dec 18, 2020

Appveyor seems to need a 3.9 wheel of zope.proxy for windows.

Appveyor has a C compiler and shouldn't need any wheels.

The error

  error: invalid command 'bdist_wheel'

seems to indicate that our appevyor.yml is missing a pip install -U wheel?

Just to see what happens.  If this works, we'll want to update the appveyor.yml template in zopefoundation/meta.
@mgedmin
Copy link
Member

mgedmin commented Dec 19, 2020

zopefoundation/meta#54 adds wheel to the appveyor.yml template.

The bug which inspired this workaround was fixed.

Co-authored-by: Marius Gedminas <marius@gedmin.as>
@sallner
Copy link
Member

sallner commented Dec 21, 2020

@mgedmin So it seems, with the exception of pypy we are all set here. This should be fixed with #38 as far as I understand. Should we merge it now, so we we can process with #38 which and other PRs?

@mgedmin
Copy link
Member

mgedmin commented Dec 22, 2020

Yes, let's.

@mgedmin mgedmin merged commit dc31ffd into master Dec 22, 2020
@mgedmin mgedmin deleted the config-with-pure-python branch December 22, 2020 15:47
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.

None yet

3 participants