-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
chore: goodbye AppVeyor, it was a good ride #7660
Conversation
ping @Daniel15, would you mind removing Jest from your AppVeyor? |
46fe73d
to
6ff89bb
Compare
Sorry I missed this! Sure, I can do that. Do we really want to delete it, though? Having the historical data can be useful. It might be worth just removing the hook from the Github repo, so that it doesn't perform any new builds? |
Removing the hook would be just enough :) |
I'd remove it, but I don't have admin access to this repo :( |
@cpojer can you remove the AppVeyor integration in the project settings? |
cc @JoelMarcey |
Ok. AppVeyor webhook deleted. |
Codecov Report
@@ Coverage Diff @@
## master #7660 +/- ##
==========================================
- Coverage 62.28% 62.27% -0.01%
==========================================
Files 264 264
Lines 10435 10435
Branches 2533 2533
==========================================
- Hits 6499 6498 -1
Misses 3354 3354
- Partials 582 583 +1
Continue to review full report at Codecov.
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
Since we set up Azure Pipelines, we there's no need for AppVeyor now. Farewell!
Test plan
cc @Daniel15, feel free to remove it from your account :)