Skip to content

ci: declare minimal permissions for Actions#9018

Merged
chenjiahan merged 2 commits intomainfrom
ci_permissions_0116
Jan 16, 2025
Merged

ci: declare minimal permissions for Actions#9018
chenjiahan merged 2 commits intomainfrom
ci_permissions_0116

Conversation

@chenjiahan
Copy link
Copy Markdown
Member

Summary

Declare minimal permissions for GitHub Actions, and remove some unused GITHUB_TOKEN.

Ref: https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/controlling-permissions-for-github_token

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions Bot added the team The issue/pr is created by the member of Rspack. label Jan 16, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 16, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 0ec68ad
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/6788baa0e3353800083cfa27

@chenjiahan chenjiahan enabled auto-merge (squash) January 16, 2025 08:00
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jan 16, 2025

CodSpeed Performance Report

Merging #9018 will not alter performance

Comparing ci_permissions_0116 (0ec68ad) with main (54ddde8)

Summary

✅ 3 untouched benchmarks

@chenjiahan chenjiahan merged commit f9aa179 into main Jan 16, 2025
@chenjiahan chenjiahan deleted the ci_permissions_0116 branch January 16, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants