Skip to content

Conversation

@sheeeng
Copy link

@sheeeng sheeeng commented Nov 6, 2025

Wait until google/go-github#3807 is being included in the next release.

Resolves #2869.


Before the change?

  • The sha_pinning_required option for actions permissions is not supported.

After the change?

  • The sha_pinning_required option actions permissions is supported.

Pull request checklist

  • Schema migrations have been created if needed (example)
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@sheeeng sheeeng force-pushed the feat/actions_permissions_sha_pinning_required branch from 6289202 to 49ad201 Compare November 8, 2025 22:31
@sheeeng sheeeng marked this pull request as ready for review November 8, 2025 22:32
@github-actions github-actions bot added the Type: Feature New feature or request label Nov 10, 2025
github.com/golangci/unconvert v0.0.0-20240309020433-c5143eacb3ed // indirect
github.com/google/go-cmp v0.6.0 // indirect
github.com/google/go-cmp v0.7.0 // indirect
github.com/google/go-github/v78 v78.0.0 // indirect
Copy link
Contributor

Choose a reason for hiding this comment

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

Unfortunately this provider is on v67 and the upgrade has not been made to get us to v78 yet.

Copy link
Author

Choose a reason for hiding this comment

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

@nickfloyd, thanks for the review. 🙇

Shall I keep this PR open while waiting for it to be upgraded?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, let's do that... I'm hopeful we'll make some good progress getting there. This is the batch where we are tracking that, please feel free to take on any of the version bumps.

We are currently trying to move toward v68 - but the breaking changes in that one are significant - all of the old project APIs were removed.

@nickfloyd nickfloyd moved this from Backlog to BLOCKED in Terraform Provider Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Feature New feature or request

Projects

Status: BLOCKED

Development

Successfully merging this pull request may close these issues.

[FEAT]: Support sha_pinning_required for Actions permissions

2 participants