Skip to content

Exempt evals directories from version checks - #12299

Merged
auto-submit[bot] merged 2 commits into
mainfrom
exempt-evals-agents
Jul 28, 2026
Merged

Exempt evals directories from version checks#12299
auto-submit[bot] merged 2 commits into
mainfrom
exempt-evals-agents

Conversation

@reidbaker-agent

@reidbaker-agent reidbaker-agent commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

This pr is blocked by #12298.

This came up in #12297. I decided landing the tool focused pr was much easier to review than stacking this work on #12285 - @reidbaker


Agent authored pr description
This PR exempts .agents and evals directories from triggering missing changelog and version update errors.

TAG=agy
CONV=24c0c9b5-7e8d-407c-ae84-a577061bd6ff

@reidbaker
reidbaker marked this pull request as ready for review July 28, 2026 15:14

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request updates the package state utility to treat the 'evals' directory as a developer-only utility directory, ensuring that changes within it do not trigger version or changelog update requirements. A corresponding unit test has been added to verify this behavior. There are no review comments to address.

@reidbaker reidbaker added the CICD Run CI/CD label Jul 28, 2026

@camsim99 camsim99 left a comment

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.

RSLGTM

@reidbaker reidbaker added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 28, 2026
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jul 28, 2026
@auto-submit

auto-submit Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/packages/12299, because This PR has not met approval requirements for merging. The PR author is not a member of flutter-hackers and needs 1 more review(s) in order to merge this PR.

  • Merge guidelines: A PR needs at least one approved review if the author is already part of flutter-hackers or two member reviews if the author is not a member of flutter-hackers before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@reidbaker reidbaker added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 28, 2026
@auto-submit
auto-submit Bot merged commit a540ae4 into main Jul 28, 2026
15 checks passed
@auto-submit
auto-submit Bot deleted the exempt-evals-agents branch July 28, 2026 20:06
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