Skip to content

chore: add if condition to pr-triage #2644

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dylan-apex
Copy link

Currently when this workflow runs, it does all of the setup and then in the triaging script it checks if the PR has already been triaged, leading to ~45 seconds of wasted runtime. We can add a condition so that the workflow only runs when the PR does not have the bot triaged label

Leaving the instructions that reference the label since it also includes the google-contributor one

Copy link

Note

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

@adk-bot
Copy link
Collaborator

adk-bot commented Aug 21, 2025

Response from ADK Triaging Agent

Hello @dylan-apex, thank you for creating this PR!

Could you please include a testing plan section in your PR to describe how you tested these changes? This will help reviewers to review your PR more efficiently.

You can find more information in our contribution guidelines. Thanks!

@dylan-apex
Copy link
Author

@hangfei You're probably the best reviewer for this PR

@hangfei
Copy link
Collaborator

hangfei commented Aug 21, 2025

@dylan-apex thanks! Is there a way for you to test this change?

@dylan-apex
Copy link
Author

dylan-apex commented Aug 21, 2025

@hangfei tested on my repo. added bot triaged tag and the workflow was skipped on edits.
Screenshot_20250820_223828_Firefox.jpg

@hangfei
Copy link
Collaborator

hangfei commented Aug 21, 2025

@xuanyang15 Could you help review this? 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