dd-trace-dotnet icon indicating copy to clipboard operation
dd-trace-dotnet copied to clipboard

Add explicit permissions to all workflows

Open andrewlock opened this issue 1 year ago • 1 comments

Summary of changes

  • Adds explicit permissions to all workflows

Reason for change

It's a best practice to explicitly specify required permissions for all workflows, and soon we'll have to

Implementation details

Worked through each workflow, trying to work out what it does and what permissions it needs

Test coverage

I wish

Other details

Please don't just 🙈 approve this, it needs 👀 because it has the potential to break at critical times otherwise! The workflows that run as part of standard PRs should be fine, but anything that's rare/release specific needs properly understanding

andrewlock avatar Jun 24 '24 08:06 andrewlock

Converted to draft, as we should check to see what the monitor comes up with before committing to this

andrewlock avatar Jun 25 '24 07:06 andrewlock