Skip to content

ci: add test workflow token permissions - #117

Merged
kitsuyui merged 1 commit into
mainfrom
ci/add-test-workflow-permissions-20260612
Jun 22, 2026
Merged

ci: add test workflow token permissions#117
kitsuyui merged 1 commit into
mainfrom
ci/add-test-workflow-permissions-20260612

Conversation

@kitsuyui

Copy link
Copy Markdown
Owner

Summary

  • Add explicit read-only contents permission to the test workflow.
  • Keep the workflow behavior unchanged while avoiding repository-default token permissions.

Verification

  • git diff --check
  • go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.12 -color .github/workflows/test.yml
  • collateral check: clean
  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo check
  • cargo test --all-targets --all-features
  • cargo llvm-cov --lcov --output-path coverage.lcov
  • cargo tree --duplicates (0 duplicates)

@github-actions

Copy link
Copy Markdown

Code Metrics Report

main (0c91c43) #117 (f468216) +/-
Coverage 68.3% 68.3% 0.0%
Code to Test Ratio 1:0.1 1:0.1 0.0
Test Execution Time 3s 6s +3s
Details
  |                     | main (0c91c43) | #117 (f468216) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          68.3% |          68.3% | 0.0% |
  |   Files             |              4 |              4 |    0 |
  |   Lines             |            550 |            550 |    0 |
  |   Covered           |            376 |            376 |    0 |
  | Code to Test Ratio  |          1:0.1 |          1:0.1 |  0.0 |
  |   Code              |            593 |            593 |    0 |
  |   Test              |             71 |             71 |    0 |
- | Test Execution Time |             3s |             6s |  +3s |

Reported by octocov

@kitsuyui
kitsuyui merged commit 3503642 into main Jun 22, 2026
5 checks passed
@kitsuyui
kitsuyui deleted the ci/add-test-workflow-permissions-20260612 branch June 22, 2026 15:27
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