Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use OIDC keys for the OSPO API #32482

Merged
merged 7 commits into from
May 1, 2024
Merged

Use OIDC keys for the OSPO API #32482

merged 7 commits into from
May 1, 2024

Conversation

tdykstra
Copy link
Contributor

@tdykstra tdykstra commented Apr 30, 2024

Copied from dotnet/docs#40674

Contributes to #32479

This wasn't in my original PR. I caught it in docs when our test failed. 

See /dotnet/docs#40676 for the corresponding PR.
See /dotnet/docs#40676 for the corresponding PR
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This LGTM @tdykstra

I updated the YML files with the permissions fix from /dotnet/docs#40676. I'd missed that in the reference PR, and we didn't catch it until we merged.

@tdykstra tdykstra merged commit bd85146 into main May 1, 2024
3 checks passed
@tdykstra tdykstra deleted the tdykstra-patch-1 branch May 1, 2024 16:09
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.

None yet

2 participants