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

bpo-39041: Add GitHub Actions support #17594

Merged
merged 12 commits into from Dec 16, 2019
Merged

bpo-39041: Add GitHub Actions support #17594

merged 12 commits into from Dec 16, 2019

Conversation

@zooba
Copy link
Member

zooba commented Dec 13, 2019

@zooba zooba changed the title Add GitHub Actions support bpo-39041: Add GitHub Actions support Dec 13, 2019
@zooba zooba added the skip news label Dec 13, 2019
@zooba zooba mentioned this pull request Dec 13, 2019
Copy link
Member

brettcannon left a comment

Should make sure to add doc-building workflow. I would also build against -latest for the OSs. And I think you might be able to use ${runs-on} as the title so it states the OS that way.

zooba added 3 commits Dec 13, 2019
@zooba

This comment has been minimized.

Copy link
Member Author

zooba commented Dec 13, 2019

I changed the VM images to "latest", but it still seems best to have static labels. I also added the coverage test run after we know the Ubuntu tests have passed, but I don't know that we'll want the extra delay.

Maybe if I split it into a separate check we can make the coverage run optional? We'll have to merge this one first though. Once I get the docs workflow in then we can do that and iterate on the coverage run.

@brettcannon

This comment has been minimized.

Copy link
Member

brettcannon commented Dec 13, 2019

Yeah, I would split the coverage run out as a separate thing as we don't rely on it to prevent coverage slippage.

zooba added 2 commits Dec 13, 2019
@zooba

This comment has been minimized.

Copy link
Member Author

zooba commented Dec 13, 2019

From my other PR (where I've enabled the builds), it looks like this:
image

zooba added 4 commits Dec 14, 2019
Disable C coverage step.
Attempt to fix Python coverage step
@zooba zooba merged commit a76ba36 into python:master Dec 16, 2019
4 checks passed
4 checks passed
Azure Pipelines PR #20191216.17 succeeded
Details
bedevere/issue-number Issue number 39041 found
Details
bedevere/news "skip news" label found
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@zooba zooba deleted the zooba:actions branch Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.