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

Switch to GitHub Actions #114

Merged
merged 3 commits into from
Jan 6, 2021
Merged

Conversation

pquentin
Copy link
Member

@pquentin pquentin commented Jan 6, 2021

I also pin requirements because the tests break with pytest 6.2 on Python 3.8+ (https://travis-ci.org/github/python-trio/pytest-trio/builds/753132828), but haven't figured out how to enable dependabot yet.

I do want to switch to ReadTheDocs PR builds, but in another pull request.

We don't support pytest 6.2 yet, and want to switch to dependabot in the
future.
`warn_on_full_buffer` support was added in PyPy, but it did not get
released yet.
@pquentin
Copy link
Member Author

pquentin commented Jan 6, 2021

Here's the new CI in action: https://github.com/pquentin/pytest-trio/actions/runs/465373561. I'll disable AppVeyor/Travis hooks if this pull request gets merged.

@oremanj
Copy link
Member

oremanj commented Jan 6, 2021

Is it necessary to pin hypothesis and pytest-cov too? I would expect pytest ~= 6.1 to work with the other requirements as previously specified

@pquentin
Copy link
Member Author

pquentin commented Jan 6, 2021

It's not necessary, I just see it as a path towards dependabot (per #98 (comment))

@pquentin pquentin merged commit c4e1fe3 into python-trio:master Jan 6, 2021
@pquentin pquentin deleted the github-actions branch January 6, 2021 08:49
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.

2 participants