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

Migrate from Azure Pipelines to GitHub Actions #13222

Merged
merged 1 commit into from
Jun 14, 2023
Merged

Conversation

zpao
Copy link
Member

@zpao zpao commented Jun 9, 2023

Beyond just a pure migration, this also:

  • Drops Node 14 usage from CI (which also removes the need for explicit npm 8 installs)
  • Removes the "old node" test, which just checked that installing on old node failed. We shouldn't need to test that
  • Consolidates the build & test workflows so we don't end up with a proliferation of workflows.

You can see this run here from my fork (in a different branch, before squashing for the PR): https://github.com/zpao/create-react-app/actions/runs/5226343675

Also, while I dropped Node 14, I didn't include Node 18. I tried but one job failed for some reason and it wasn't worth figuring out. It's trivial to add with the matrix setup.

Beyond just a pure migration, this also:

- Drops Node 14 usage from CI (which also removes the need for explicit
  npm 8 installs)
- Removes the "old node" test, which just checked that installing on old
  node failed. We shouldn't need to test that
- Consolidates the build & test workflows so we don't end up with
  a proliferation of workflows.
@zpao
Copy link
Member Author

zpao commented Jun 14, 2023

Azure pipelines is no longer running, so I'm going to land this for some parity (and improvement). FWIW I am not stepping up to maintain this project - this is a one-off cleanup to move off of Azure.

@zpao zpao merged commit 0a827f6 into facebook:main Jun 14, 2023
@ZxSix666
Copy link

bb

unional pushed a commit to unional/create-react-app that referenced this pull request Feb 21, 2024
Beyond just a pure migration, this also:

- Drops Node 14 usage from CI (which also removes the need for explicit
  npm 8 installs)
- Removes the "old node" test, which just checked that installing on old
  node failed. We shouldn't need to test that
- Consolidates the build & test workflows so we don't end up with
  a proliferation of workflows.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants