Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Move CI to GitHub Actions #7903

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 5 commits into from
Apr 6, 2021
Merged

Conversation

adamchainz
Copy link
Contributor

  • Recreate all the jobs on GitHub Actions
  • Upgrade to Ubuntu 20.04
  • Upgrade base/docs/dist to Python 3.9

* Recreate all the jobs on GitHub Actions
* Upgrade to Ubuntu 20.04
* Upgrade base/docs/dist to Python 3.9
[testenv:py37-django32]
ignore_errors = true

[testenv:py38-django32]
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we be allowing test failures on 3.2? It's due this week?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just copying what's there already - I think there's a process to update things when the release comes out.

@adamchainz
Copy link
Contributor Author

Alright we are green! ✅

Copy link
Collaborator

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

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

Yay! Thanks for this @adamchainz ⛵️

(What is this Python setup that doesn't have wheel? 🤔😀)

@tomchristie
Copy link
Member

This is fantastic, thanks Adam!

It's worth comparing and contrasting with our HTTPX workflow, here... https://github.com/encode/httpx/blob/master/.github/workflows/test-suite.yml

I'm pretty keen on the style in the HTTPX approach of GitHub's scripts to rule them all pattern, together with one-liner steps.

Also different there is that we don't currently use caching, and if we did, we'd introduce it in a follow-up commit, to help us review the UI difference that it introduces, so we can better understand where it fits in, and be in a better position to assess if something changes/breaks there.

Having said both of those things I'm completely fine with you taking a lead on this @adamchainz - we can either...

  • Merge this as is, keep possible HTTPX workflow style follow ups as later incremental PRs.
  • Merge this as is, and we're happy with it.
  • Introduce some updates to this PR based on the HTTPX workflow style.

For myself, I'mma approve this, and then leave that in your court.

@adamchainz
Copy link
Contributor Author

I'm personally not that sold on the "scripts for everything" approach. I think it removes some of the power of the CI environment, such as caching, running pre-commit in parallel through its action, etc.

Would be happy to convert django-rest-framework to that pattern in a follow-up though if it's really desired.

@adamchainz adamchainz merged commit fd017d0 into encode:master Apr 6, 2021
@adamchainz adamchainz deleted the github_actions branch April 6, 2021 17:34
@adamchainz adamchainz mentioned this pull request Apr 6, 2021
terencehonles added a commit to terencehonles/django-rest-framework that referenced this pull request Apr 7, 2021
terencehonles added a commit to terencehonles/django-rest-framework that referenced this pull request Apr 7, 2021
adamchainz added a commit to adamchainz/django-rest-framework that referenced this pull request Apr 16, 2021
The config I copied in encode#7903 was from a repo with the new name 'main', so tests have not been running on master since.
adamchainz added a commit that referenced this pull request Apr 16, 2021
The config I copied in #7903 was from a repo with the new name 'main', so tests have not been running on master since.
sigvef pushed a commit to sigvef/django-rest-framework that referenced this pull request Dec 3, 2022
* Recreate all the jobs on GitHub Actions
* Upgrade to Ubuntu 20.04
* Upgrade base/docs/dist to Python 3.9
sigvef pushed a commit to sigvef/django-rest-framework that referenced this pull request Dec 3, 2022
sigvef pushed a commit to sigvef/django-rest-framework that referenced this pull request Dec 3, 2022
The config I copied in encode#7903 was from a repo with the new name 'main', so tests have not been running on master since.
@adamchainz adamchainz mentioned this pull request Dec 8, 2022
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.

4 participants