Skip to content

Conversation

@osterman
Copy link
Member

what

  • Added tj-actions/changed-files step to detect if .github/CODEOWNERS was modified in the PR
  • Only run CODEOWNERS validation checks when the file actually changed or on manual workflow_dispatch triggers
  • Prevents unnecessary validation overhead for PRs that don't touch the CODEOWNERS file

why

  • The CODEOWNERS validation should only run when the file is actually modified
  • This reduces CI overhead and prevents friction for unrelated PRs
  • Always allow manual validation via workflow_dispatch for verification purposes

references

  • Uses tj-actions/changed-files@v45 - a widely-used, well-maintained action for detecting changed files

Use tj-actions/changed-files to detect if .github/CODEOWNERS was modified in the PR.
Only run validation checks when the CODEOWNERS file actually changed or on manual workflow_dispatch.
This prevents unnecessary validation overhead for PRs that don't touch CODEOWNERS.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@osterman osterman requested a review from a team as a code owner January 23, 2026 19:21
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

Warning

Rate limit exceeded

@osterman has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 40 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch osterman/codeowners-check-skip

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@osterman osterman added the no-release Do not create a new release (wait for additional code changes) label Jan 23, 2026
@github-actions github-actions bot added the size/s Small size PR label Jan 23, 2026
@mergify
Copy link

mergify bot commented Jan 23, 2026

Important

Cloud Posse Engineering Team Review Required

This pull request modifies files that require Cloud Posse's review. Please be patient, and a core maintainer will review your changes.

To expedite this process, reach out to us on Slack in the #pr-reviews channel.

@mergify mergify bot added the needs-cloudposse Needs Cloud Posse assistance label Jan 23, 2026
@github-actions
Copy link

github-actions bot commented Jan 23, 2026

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

  • .github/workflows/validate-codeowners.yml

@osterman osterman enabled auto-merge (squash) January 23, 2026 19:31
@codecov
Copy link

codecov bot commented Jan 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.35%. Comparing base (34d6675) to head (ca12c84).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2016      +/-   ##
==========================================
- Coverage   75.36%   75.35%   -0.01%     
==========================================
  Files         792      792              
  Lines       73396    73396              
==========================================
- Hits        55315    55310       -5     
- Misses      14570    14578       +8     
+ Partials     3511     3508       -3     
Flag Coverage Δ
unittests 75.35% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@osterman osterman merged commit 50414bc into main Jan 23, 2026
64 checks passed
@osterman osterman deleted the osterman/codeowners-check-skip branch January 23, 2026 19:47
@mergify mergify bot removed the needs-cloudposse Needs Cloud Posse assistance label Jan 23, 2026
@github-actions
Copy link

These changes were released in v1.204.1-rc.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-release Do not create a new release (wait for additional code changes) size/s Small size PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants