Skip to content

Conversation

@yasu89
Copy link
Owner

@yasu89 yasu89 commented Jun 21, 2025

This pull request includes a small change to the .github/workflows/tagpr.yml file. The change adds specific permissions (packages: write, contents: read, attestations: write, id-token: write) to the workflow configuration for better control and security.

Copilot AI review requested due to automatic review settings June 21, 2025 18:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the workflow configuration to explicitly set job permissions for improved security during the docker-release process.

  • Explicit permissions added to the job configuration.
  • Enhances control over package, attestation, and id-token operations.

@yasu89 yasu89 merged commit 94bc645 into main Jun 21, 2025
2 checks passed
@yasu89 yasu89 deleted the fix/docker-image-permission branch June 21, 2025 18:59
@github-actions github-actions bot mentioned this pull request Jun 21, 2025
@github-actions
Copy link
Contributor

Code Metrics Report

main (076d822) #18 (c62b5f9) +/-
Coverage 69.6% 69.6% 0.0%
Code to Test Ratio 1:1.7 1:1.7 0.0
Test Execution Time 26s 27s +1s
Details
  |                     | main (076d822) | #18 (c62b5f9) | +/-  |
  |---------------------|----------------|---------------|------|
  | Coverage            |          69.6% |         69.6% | 0.0% |
  |   Files             |              3 |             3 |    0 |
  |   Lines             |             99 |            99 |    0 |
  |   Covered           |             69 |            69 |    0 |
  | Code to Test Ratio  |          1:1.7 |         1:1.7 |  0.0 |
  |   Code              |            206 |           206 |    0 |
  |   Test              |            353 |           353 |    0 |
- | Test Execution Time |            26s |           27s |  +1s |

Reported by octocov

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.

2 participants