Skip to content

ci: ignore Renovate bot on irrelevant jobs#2904

Merged
gauthier-th merged 1 commit intodevelopfrom
ci/ignore-renovate
Apr 20, 2026
Merged

ci: ignore Renovate bot on irrelevant jobs#2904
gauthier-th merged 1 commit intodevelopfrom
ci/ignore-renovate

Conversation

@M0NsTeRRR
Copy link
Copy Markdown
Member

@M0NsTeRRR M0NsTeRRR commented Apr 18, 2026

Description

n/a

How Has This Been Tested?

n/a

Screenshots / Logs (if applicable)

Checklist:

  • I have read and followed the contribution guidelines.
  • Disclosed any use of AI (see our policy)
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required)

Summary by CodeRabbit

  • Chores
    • Optimized CI workflows to skip certain validation and labeling steps when runs are triggered by automated dependency updates, reducing unnecessary checks.
    • Minor workflow metadata formatting and end-of-file formatting adjustments.

@M0NsTeRRR M0NsTeRRR requested a review from a team as a code owner April 18, 2026 15:13
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2f059970-0a90-4602-94b8-0f3a03457ca1

📥 Commits

Reviewing files that changed from the base of the PR and between 1832606 and aa4a734.

📒 Files selected for processing (2)
  • .github/workflows/conflict_labeler.yml
  • .github/workflows/pr-validation.yml
🚧 Files skipped from review as they are similar to previous changes (2)
  • .github/workflows/conflict_labeler.yml
  • .github/workflows/pr-validation.yml

📝 Walkthrough

Walkthrough

Added job-level conditions to two GitHub Actions workflows so specific jobs skip execution when the run is triggered by renovate[bot]; also adjusted workflow name quoting and added a trailing newline in one file.

Changes

Cohort / File(s) Summary
Conflict labeler workflow
​.github/workflows/conflict_labeler.yml
Added if: github.actor != 'renovate[bot]' at the job level to prevent the labeling job (and its steps) from running for renovate[bot].
PR validation workflow
​.github/workflows/pr-validation.yml
Expanded template-check job if to also skip when github.actor == 'renovate[bot]'; changed workflow name quotes and added EOF newline.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • fallenbagel
  • gauthier-th

Poem

🐇 I hopped through lines of YAML bright,

Skipped the bot that works by night,
Labels paused and checks go light,
Quiet branches sleep polite. 🌙

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding conditions to skip Renovate bot on irrelevant CI jobs in two workflow files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

@M0NsTeRRR M0NsTeRRR changed the title ci: ignore Renovate updates for irrelevant jobs ci: ignore Renovate bot on irrelevant jobs Apr 18, 2026
Comment thread .github/workflows/pr-validation.yml Fixed
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/pr-validation.yml:
- Line 113: The if: condition is being treated as a non-empty string because it
mixes a nested `${{ }}` inside an implicit expression and uses the invalid
operator `and`; update the condition to remove the nested `${{ }}` and use the
GitHub Actions boolean operator `&&` so it reads as an expression comparing
github.event.action != 'synchronize' and github.actor != 'renovate[bot]' (refer
to the existing if: line and the symbols github.event.action, github.actor,
'synchronize', and 'renovate[bot]').
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 73f8c559-8777-4488-9dae-0206f59b8880

📥 Commits

Reviewing files that changed from the base of the PR and between 56ab94a and 1832606.

📒 Files selected for processing (2)
  • .github/workflows/conflict_labeler.yml
  • .github/workflows/pr-validation.yml

Comment thread .github/workflows/pr-validation.yml Outdated
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
@M0NsTeRRR M0NsTeRRR force-pushed the ci/ignore-renovate branch from 1832606 to aa4a734 Compare April 18, 2026 15:16
@seerr-automation-bot seerr-automation-bot added this to the v3.3.0 milestone Apr 18, 2026
@gauthier-th gauthier-th merged commit 1d2aa76 into develop Apr 20, 2026
17 checks passed
@gauthier-th gauthier-th deleted the ci/ignore-renovate branch April 20, 2026 08:37
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.

5 participants