Skip to content

Remove permission#49

Merged
maryliag merged 3 commits into
open-telemetry:mainfrom
maryliag:remove-permission
Jul 8, 2026
Merged

Remove permission#49
maryliag merged 3 commits into
open-telemetry:mainfrom
maryliag:remove-permission

Conversation

@maryliag

@maryliag maryliag commented Jul 8, 2026

Copy link
Copy Markdown
Member

This permission was not being used and called error on callers

@maryliag maryliag requested a review from a team as a code owner July 8, 2026 15:57
Copilot AI review requested due to automatic review settings July 8, 2026 15:57
@maryliag maryliag enabled auto-merge (squash) July 8, 2026 16:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR updates the first-time-pr.yml GitHub Actions workflow to remove an unused contents: read permission and replace it with an empty permissions block. According to the description, the previous permission was unused and was causing errors on callers.

Changes:

  • Replaced permissions:\n contents: read with permissions: {} to explicitly grant no permissions to the workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@maryliag maryliag disabled auto-merge July 8, 2026 16:23
@maryliag maryliag merged commit a4036f5 into open-telemetry:main Jul 8, 2026
5 checks passed
@maryliag maryliag deleted the remove-permission branch July 8, 2026 16:23
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.

3 participants