-
Notifications
You must be signed in to change notification settings - Fork 110
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
TST: Switch to Visual Studio 2022 for Appveyor Windows setup #6228
Conversation
This is the latest build worker image available, and comes with Git version 2.33.1, which could be useful in switching test setups to not overwrite USERPROFILE environment variables (cc #6160)
ugh, crap - I didn't PR from a fork 🤦 sorry. I'll let appveyor finish anyway. |
I killed the branch CI too (one is enough to tell us if it works). Thanks, on behalf of the planet, for canceling all the rest! |
Codecov Report
@@ Coverage Diff @@
## master #6228 +/- ##
===========================================
- Coverage 86.19% 64.25% -21.95%
===========================================
Files 323 323
Lines 41849 41832 -17
===========================================
- Hits 36072 26878 -9194
- Misses 5777 14954 +9177
Continue to review full report at Codecov.
|
Massive failure -- but looks homogeneous at least |
Fancy failures.
There is a total of whooping 81 Windows failures that I previously reported in #5871 -- the failure is at the stage of git annex init, and should it be the same as in the linked issue, it arises from a super weird test setup in which the main branch is not a valid SHA1. We've previously thought the cause of this was in the git-annex version, but the errors here happened with our standard appveyor git-annex version which was thought to circumvent those fixes.
As far as I can see atm, the test repos are in a state that is uninitializable. |
Works! Let's run with this! Thx! |
This is the latest build worker image available, and comes with Git
version 2.33.1, which could be useful in switching test setups to
not overwrite USERPROFILE environment variables (cc #6160)
Just a test PR to check whether everything works out of the box.