-
-
Notifications
You must be signed in to change notification settings - Fork 852
cut down appveyor's env matrix since we're now running serially #9895
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
Conversation
Deleted the .github/workflows/claude-code-review.yml file, removing the automated Claude code review workflow from the repository.
Deleted the .github/workflows/claude-code-review.yml file to disable the automated Claude code review process on pull requests.
|
@claude please research the appveyor failures. is it resource contention? did the test end up on the wrong matrix? |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
@claude try again |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
i mightve run out of tokens 🤔 |
|
@claude what does it mean? It's on niph's repo. Can you go "get it" like github desktop?
|
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
@claude try again the branch is now |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
FINE -- @niphlod ill have to report this as a bug, claude cannot check out your branches on your repo :( I'll create a new one since I'm very curious what the problem could be. |

This cuts the appveyor matrix in half.
We were splitting scenarios in two parts to leverage appveyor's parallelism, but we're now restricted to one concurrent job, so there's no reason to run splitted parts.
On my test runs, it seems to break for Invoke-DbaDbMirroring on 2016_2017 pretty consistently, let's see how it goes in here.