Skip to content

CI: delete AppVeyor config#25373

Merged
kassens merged 1 commit into
react:mainfrom
kassens:appveyor
Sep 30, 2022
Merged

CI: delete AppVeyor config#25373
kassens merged 1 commit into
react:mainfrom
kassens:appveyor

Conversation

@kassens

@kassens kassens commented Sep 30, 2022

Copy link
Copy Markdown
Contributor

I noticed this file as #24892 is updating the NodeJS version in it.

I don't think this is used for anything as the config just runs a handful of yarn commands and I assume that's all covered by CircleCI.

@kassens
kassens requested review from acdlite and bvaughn September 30, 2022 22:02
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Sep 30, 2022
I noticed this file as react#24892 is updating the NodeJS version in it.

I don't think this is used for anything as the config just runs a handful
of yarn commands and I assume that's all covered by CircleCI.

@bvaughn bvaughn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

TBH I don't really know what this file is being used for. Delete it and just revert if something breaks? 🥳

@kassens
kassens merged commit d061e6e into react:main Sep 30, 2022
@kassens
kassens deleted the appveyor branch September 30, 2022 23:02
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
I noticed this file as react#24892 is updating the NodeJS version in it.

I don't think this is used for anything as the config just runs a handful
of yarn commands and I assume that's all covered by CircleCI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants