Skip to content

Conversation

@niphlod
Copy link
Contributor

@niphlod niphlod commented Oct 30, 2025

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.

niphlod and others added 4 commits October 29, 2025 22:20
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.
@potatoqualitee
Copy link
Member

@claude please research the appveyor failures. is it resource contention? did the test end up on the wrong matrix?

@claude
Copy link

claude bot commented Oct 31, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@potatoqualitee
Copy link
Member

@claude try again

@claude
Copy link

claude bot commented Oct 31, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@potatoqualitee
Copy link
Member

i mightve run out of tokens 🤔

@potatoqualitee
Copy link
Member

@claude what does it mean?

Mode: tag
Context prompt: NO PROMPT
Trigger result: true
Preparing with mode: tag for event: issue_comment
Actor type: User
Verified human actor: potatoqualitee
✅ Created initial comment with ID: 3471607335
Successfully fetched PR #9895 data
This is an open PR, checking out PR branch...
PR #9895: 4 commits, using fetch depth 20
fatal: couldn't find remote ref tests/appveyor_matrix
Error: Prepare step failed with error: Failed with exit code 128
Error: Process completed with exit code 1.
Run bun run ${GITHUB_ACTION_PATH}/src/entrypoints/update-comment-link.ts
Fetching issue comment 3471607335
Successfully fetched as issue comment
✅ Updated issue comment 3471607335 with job link

It's on niph's repo. Can you go "get it" like github desktop?

image

@claude
Copy link

claude bot commented Oct 31, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@potatoqualitee
Copy link
Member

@claude try again the branch is now tests/appveyor_matrix_niphlod

@claude
Copy link

claude bot commented Oct 31, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@potatoqualitee
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants