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

Add Azure DevOps build pipeline #7096

Merged
merged 26 commits into from
May 29, 2019
Merged

Add Azure DevOps build pipeline #7096

merged 26 commits into from
May 29, 2019

Conversation

ianschmitz
Copy link
Contributor

@ianschmitz ianschmitz commented May 22, 2019

This PR adds support to run our CI builds using Azure DevOps. Huge thanks to @willsmythe and others at Microsoft for their help getting us moved over.

TODO:

@petetnt
Copy link
Contributor

petetnt commented May 22, 2019

Should we test on Node 12 too now that it is out?

@iansu
Copy link
Contributor

iansu commented May 22, 2019

@petetnt Do the tests actually pass on Node 12?

We should start working on getting everything working on Node 12 because people are already using it. It won't be LTS until October but we definitely want to have it working before then. That's going to add something like 11 more checks though. 😬

@petetnt
Copy link
Contributor

petetnt commented May 23, 2019

@iansu haven't tested yet myself, but I did see that there were some issues with CRA / its dependencies on Node@12 at some point IIRC

@ianschmitz ianschmitz self-assigned this May 26, 2019
test/fixtures/mjs-support/src/App.js Show resolved Hide resolved
packages/react-scripts/scripts/utils/createJestConfig.js Outdated Show resolved Hide resolved
tasks/e2e-simple.sh Outdated Show resolved Hide resolved
test/fixtures/__shared__/util/scripts.js Show resolved Hide resolved
@ianschmitz ianschmitz changed the title WIP: Add Azure DevOps build pipeline Add Azure DevOps build pipeline May 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants