Skip to content

Use pull-request trigger for block dart skills prs - #251

Merged
chunhtai merged 1 commit into
flutter:mainfrom
chunhtai:fix-security
Oct 26, 2026
Merged

chunhtai merged 1 commit into
flutter:mainfrom
chunhtai:fix-security

Conversation

@chunhtai

Copy link
Copy Markdown
Contributor

b/577108809

The change from commenting on PR to fails the presubmit check. The downside is a PR can by pass this by updating the workflow file, but reviewer should be able to catch it

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@chunhtai
chunhtai requested a review from keertip October 23, 2026 23:16
@chunhtai
chunhtai merged commit 90fa51d into flutter:main Oct 26, 2026
11 checks passed
@keertip

keertip commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator

@chunhtai, thanks!

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.

3 participants