Skip to content

Conversation

@utpilla
Copy link
Contributor

@utpilla utpilla commented Apr 7, 2025

Changes

  • Add permissions as a top-level key in all workflows

Security Fixes

Least Privileged GitHub Actions Token Permissions

The GITHUB_TOKEN is an automatically generated secret to make authenticated calls to the GitHub API. GitHub recommends setting minimum token permissions for the GITHUB_TOKEN.

@codecov
Copy link

codecov bot commented Apr 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.3%. Comparing base (9a0ffc4) to head (7ded19f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #2911     +/-   ##
=======================================
- Coverage   81.3%   81.3%   -0.1%     
=======================================
  Files        126     126             
  Lines      24156   24156             
=======================================
- Hits       19650   19649      -1     
- Misses      4506    4507      +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@utpilla utpilla closed this Apr 7, 2025
@utpilla utpilla reopened this Apr 7, 2025
@utpilla utpilla added the semver-check For PRs that need to run semver compliance checks label Apr 7, 2025
@utpilla utpilla changed the title fix: Harden GitHub Actions ci: Harden GitHub Actions Apr 7, 2025
@utpilla utpilla marked this pull request as ready for review April 7, 2025 21:31
@utpilla utpilla requested a review from a team as a code owner April 7, 2025 21:31
@utpilla utpilla merged commit 940ec23 into open-telemetry:main Apr 7, 2025
50 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance semver-check For PRs that need to run semver compliance checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants