Skip to content

ci: issues permissions for changeset preview#10284

Merged
birkskyum merged 1 commit intomainfrom
ci--issues-permissions-on-changeset-preview
Mar 18, 2026
Merged

ci: issues permissions for changeset preview#10284
birkskyum merged 1 commit intomainfrom
ci--issues-permissions-on-changeset-preview

Conversation

@birkskyum
Copy link
Member

@birkskyum birkskyum commented Mar 18, 2026

the changeset preview can't post in the PR the expected bumps - it's because it lacks issues write access

Summary by CodeRabbit

  • Chores
    • Updated GitHub workflow permissions to enhance issue management capabilities.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 703d3290-724f-4344-ba19-94de66ec1b54

📥 Commits

Reviewing files that changed from the base of the PR and between 84a8ff1 and 90bec60.

📒 Files selected for processing (1)
  • .github/workflows/pr.yml

📝 Walkthrough

Walkthrough

A single line addition to the GitHub workflow configuration file that grants the workflow write access to issues, expanding its permissions while maintaining existing access for contents and pull-requests.

Changes

Cohort / File(s) Summary
GitHub Workflow Configuration
.github/workflows/pr.yml
Added write permission for issues resource to the workflow permissions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A permission so grand, now the workflow can write,
To issues it whispers, with newfound might,
One line of magic in YAML so fine,
Now issues shall tremble at each CI sign! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete. While it explains the issue, it lacks the required template structure with sections for Changes, Checklist, and Release Impact. Complete the PR description using the repository template: add detailed explanation under Changes section, check the Contributing guide and test checklist items, and specify whether this is a CI-only change under Release Impact.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding issues write permissions to the CI workflow for the changeset preview functionality.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 ci--issues-permissions-on-changeset-preview
📝 Coding Plan
  • Generate coding plan for human review comments

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

nx-cloud bot commented Mar 18, 2026

View your CI Pipeline Execution ↗ for commit 90bec60

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

☁️ Nx Cloud last updated this comment at 2026-03-18 01:58:11 UTC

@github-actions
Copy link
Contributor

🚀 Changeset Version Preview

1 package(s) bumped directly, 0 bumped as dependents.

🟩 Patch bumps

Package Version Reason
@tanstack/eslint-plugin-query 5.91.4 → 5.91.5 Changeset

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 18, 2026

More templates

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@10284

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@10284

@tanstack/preact-query

npm i https://pkg.pr.new/@tanstack/preact-query@10284

@tanstack/preact-query-devtools

npm i https://pkg.pr.new/@tanstack/preact-query-devtools@10284

@tanstack/preact-query-persist-client

npm i https://pkg.pr.new/@tanstack/preact-query-persist-client@10284

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@10284

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@10284

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@10284

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@10284

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@10284

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@10284

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@10284

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@10284

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@10284

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@10284

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@10284

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@10284

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@10284

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@10284

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@10284

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@10284

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@10284

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@10284

commit: 90bec60

@github-actions
Copy link
Contributor

size-limit report 📦

Path Size
react full 11.92 KB (0%)
react minimal 8.95 KB (0%)

@birkskyum birkskyum merged commit e924517 into main Mar 18, 2026
7 of 8 checks passed
@birkskyum birkskyum deleted the ci--issues-permissions-on-changeset-preview branch March 18, 2026 01:58
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