Skip to content

fix Error: Could not queue the build because there were validation errors or warnings.#128

Merged
jakebailey merged 1 commit intomicrosoft:masterfrom
iisaduan:goof
Apr 1, 2026
Merged

fix Error: Could not queue the build because there were validation errors or warnings.#128
jakebailey merged 1 commit intomicrosoft:masterfrom
iisaduan:goof

Conversation

@iisaduan
Copy link
Copy Markdown
Member

@iisaduan iisaduan commented Mar 31, 2026

When attempting to sanitize the debug logging, I'm pretty sure I found the bug

closes #127

Comment on lines -116 to -120
if (info.tsgo) {
parameters.source_owner = info.owner;
parameters.source_repo = info.repo;
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I put this in #123 on purpose because this also breaks unrelated jobs like bumping the release, cherry picking, etc.

Copy link
Copy Markdown
Member Author

@iisaduan iisaduan Mar 31, 2026

Choose a reason for hiding this comment

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

I'm trying to delete info.owner and info.repo from being passed into the parameters altogether

Copy link
Copy Markdown
Member Author

@iisaduan iisaduan Mar 31, 2026

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

If the specific pipeline needs info.owner or info.repo, they should be passed in via inputs and copied over in the call to Object.assign. (We also don't have any pipelines that need those params anyways)

@iisaduan iisaduan requested a review from jakebailey March 31, 2026 22:57
@jakebailey jakebailey merged commit 04e5612 into microsoft:master Apr 1, 2026
2 checks passed
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