Skip to content

chore: only run autofix on PRs#333

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

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

Conversation

@KevinVandy
Copy link
Copy Markdown
Member

@KevinVandy KevinVandy commented May 21, 2026

🎯 Changes

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm 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 GitHub Actions workflow configuration for continuous integration processes.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 96f79366-516f-4572-b7f2-35cb9ed56a79

📥 Commits

Reviewing files that changed from the base of the PR and between 09139b1 and 807f5b6.

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

📝 Walkthrough

Walkthrough

The pull request modifies the GitHub Actions workflow trigger configuration in .github/workflows/autofix.yml. The workflow previously ran on push events to main, alpha, and beta branches; it now runs on pull_request events. Workflow jobs and other configuration remain unchanged.

Changes

Autofix Workflow Trigger Update

Layer / File(s) Summary
Workflow trigger configuration
.github/workflows/autofix.yml
The on trigger is changed from a push event with branch filters (main, alpha, beta) to a pull_request event trigger.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A workflow now dances on pull requests instead,
No more pushing to branches, the trigger has fled!
Simple, swift changes make the CI flow bright,
Autofix now listens when reviews take flight. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description follows the required template structure with all sections present, but the 'Changes' section contains only a comment placeholder with no actual description of what was changed or why. Fill in the 'Changes' section with a concrete description of the workflow modification and its motivation for this configuration change.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: restricting the autofix workflow to run only on pull requests instead of on pushes to specific branches.
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 autofix-config

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 807f5b6

Command Status Duration Result
nx affected --targets=test:sherif,test:knip,tes... ✅ Succeeded 27s View ↗
nx run-many --target=build --exclude=examples/** ✅ Succeeded 1s View ↗

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 21, 2026

@tanstack/angular-store

npm i https://pkg.pr.new/@tanstack/angular-store@333

@tanstack/lit-store

npm i https://pkg.pr.new/@tanstack/lit-store@333

@tanstack/preact-store

npm i https://pkg.pr.new/@tanstack/preact-store@333

@tanstack/react-store

npm i https://pkg.pr.new/@tanstack/react-store@333

@tanstack/solid-store

npm i https://pkg.pr.new/@tanstack/solid-store@333

@tanstack/store

npm i https://pkg.pr.new/@tanstack/store@333

@tanstack/svelte-store

npm i https://pkg.pr.new/@tanstack/svelte-store@333

@tanstack/vue-store

npm i https://pkg.pr.new/@tanstack/vue-store@333

commit: 807f5b6

@KevinVandy KevinVandy merged commit 2319b86 into main May 21, 2026
10 checks passed
@KevinVandy KevinVandy deleted the autofix-config branch May 21, 2026 22:08
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.

1 participant