Skip to content

Bubble up more descriptive logs from GitHub when enqueuing a PR fails.#5017

Merged
auto-submit[bot] merged 1 commit intoflutter:mainfrom
eyebrowsoffire:graphql_logging
Apr 8, 2026
Merged

Bubble up more descriptive logs from GitHub when enqueuing a PR fails.#5017
auto-submit[bot] merged 1 commit intoflutter:mainfrom
eyebrowsoffire:graphql_logging

Conversation

@eyebrowsoffire
Copy link
Copy Markdown
Contributor

GitHub does provide useful and descriptive messages when there is some issue with enqueuing the PR into the merge queue. Unfortunately, right now we're just saying 400 GraphQL mutate failed which doesn't help the user figure out how to unblock their PR. This change adds the info from the GitHub response in to the message.

@eyebrowsoffire eyebrowsoffire added the CICD Run CI/CD label Apr 8, 2026
Copy link
Copy Markdown
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

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

LGTM

graphqlErrors: <GraphQLError>[
const GraphQLError(
message:
'Pull request New changes require approval from someone other than Piinks because they were the last pusher.',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lol, bad Piinks.

@eyebrowsoffire eyebrowsoffire added the autosubmit Merge PR when tree becomes green via auto submit App. label Apr 8, 2026
@auto-submit auto-submit bot merged commit 6e04c32 into flutter:main Apr 8, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App. CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants