Skip to content

chore: only run autofix on PRs#225

Merged
KevinVandy merged 1 commit into
mainfrom
config-autofix
May 21, 2026
Merged

chore: only run autofix on PRs#225
KevinVandy merged 1 commit into
mainfrom
config-autofix

Conversation

@KevinVandy
Copy link
Copy Markdown
Member

@KevinVandy KevinVandy commented May 21, 2026 •
edited by coderabbitai Bot
Loading

🎯 Changes

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow configuration to adjust automation triggers.

Note: This release contains no user-facing changes.

Review Change Stack

@KevinVandy KevinVandy requested a review from a team as a code owner May 21, 2026 21:21
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6c76a120-1070-4dd5-a34d-c359169ed91c

📥 Commits

Reviewing files that changed from the base of the PR and between a894009 and 6f7a944.

📒 Files selected for processing (1)
  • .github/workflows/autofix.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/autofix.yml
📝 Walkthrough

Walkthrough

This PR removes the push event trigger from the GitHub Actions autofix workflow, leaving pull_request as the only activation event. The workflow configuration is simplified by eliminating the branch-specific push trigger for main, alpha, beta, and rc.

Changes

Autofix Workflow Trigger

Layer / File(s) Summary
Remove push trigger from autofix workflow
.github/workflows/autofix.yml		 The workflow on: section removes the push trigger (including its branch list) and retains pull_request as the only event that activates the autofix workflow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 The autofix now runs with pull requests so fair,
No more pushing to branches scattered here and there,
Simpler config, cleaner ways,
Fewer triggers, brighter days!
A workflow refined with care. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description uses the required template structure but the 'Changes' section is completely empty, lacking any explanation of what was changed or the motivation. Fill in the 'Changes' section with a description of the workflow modification and why running autofix only on PRs is beneficial.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: removing the push trigger from autofix workflow to run only on pull requests.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch config-autofix

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.

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented May 21, 2026

View your CI Pipeline Execution ↗ for commit 6f7a944

Command Status Duration Result
nx run-many --targets=build --exclude=examples/** ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-05-21 21:22:23 UTC

1 similar comment
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented May 21, 2026

View your CI Pipeline Execution ↗ for commit 6f7a944

Command Status Duration Result
nx run-many --targets=build --exclude=examples/** ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-05-21 21:22:23 UTC

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 21, 2026

More templates

@tanstack/angular-pacer

npm i https://pkg.pr.new/@tanstack/angular-pacer@225

@tanstack/pacer

npm i https://pkg.pr.new/@tanstack/pacer@225

@tanstack/pacer-devtools

npm i https://pkg.pr.new/@tanstack/pacer-devtools@225

@tanstack/pacer-lite

npm i https://pkg.pr.new/@tanstack/pacer-lite@225

@tanstack/preact-pacer

npm i https://pkg.pr.new/@tanstack/preact-pacer@225

@tanstack/preact-pacer-devtools

npm i https://pkg.pr.new/@tanstack/preact-pacer-devtools@225

@tanstack/react-pacer

npm i https://pkg.pr.new/@tanstack/react-pacer@225

@tanstack/react-pacer-devtools

npm i https://pkg.pr.new/@tanstack/react-pacer-devtools@225

@tanstack/solid-pacer

npm i https://pkg.pr.new/@tanstack/solid-pacer@225

@tanstack/solid-pacer-devtools

npm i https://pkg.pr.new/@tanstack/solid-pacer-devtools@225

commit: 6f7a944

@KevinVandy KevinVandy merged commit 9467302 into main May 21, 2026
9 checks passed
@KevinVandy KevinVandy deleted the config-autofix branch May 21, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schiller-manuel schiller-manuel schiller-manuel approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KevinVandy @schiller-manuel