Skip to content

Add GitHub actions for CI #6355

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

Merged
merged 1 commit into from
Dec 26, 2019
Merged

Conversation

nicoddemus
Copy link
Member

@nicoddemus nicoddemus commented Dec 18, 2019

This includes our current full matrix (windows, linux and macos), for evaluting
purposes.

We should disconsider failures when evaluating PRs.

TODO:

  • deploy
  • coverage
  • github release notes

Even with the above missing, I still believe it would be nice to merge
this and have GitHub actions working in parallel so we can evaluate performance
and usability from now on.

Unfortunately due to the way GitHub actions work, the jobs are not triggered until GitHub actions are enabled on master, but the results of this build can be seen here:

nicoddemus#11

Copy link
Contributor

@samueljsb samueljsb left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Is it worth adding MacOS for python 3.8 while you're here?

This includes our current full matrix (windows, linux and macos), for evaluting
purposes.

We should disconsider failures when evaluating PRs.

TODO:

- deploy
- coverage
- github release notes

Even with the above missing, I still believe it would be nice to merge
this and have GitHub actions working in parallel so we can evaluate performance
and usability from now on.
@nicoddemus
Copy link
Member Author

Is it worth adding MacOS for python 3.8 while you're here?

Done! It is now building, hopefully MacOS Python 3.8 will also pass. 🤞

@nicoddemus nicoddemus merged commit 06206bc into pytest-dev:master Dec 26, 2019
@nicoddemus nicoddemus deleted the add-gh-actions branch December 26, 2019 16:32
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