Skip to content

docs: add example on how to grant security manager role with github_organization_role_team resource#3031

Merged
stevehipwell merged 3 commits intointegrations:mainfrom
KyriosGN0:sec-manager
Feb 17, 2026
Merged

docs: add example on how to grant security manager role with github_organization_role_team resource#3031
stevehipwell merged 3 commits intointegrations:mainfrom
KyriosGN0:sec-manager

Conversation

@KyriosGN0
Copy link
Contributor

@KyriosGN0 KyriosGN0 commented Jan 1, 2026

Resolves #2981


Before the change?

  • no example on how to grant security manager role with the github_organization_tole_team resource

After the change?

  • example exist

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

@github-actions
Copy link

github-actions bot commented Jan 1, 2026

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

Copy link
Member

@nickfloyd nickfloyd left a comment

Choose a reason for hiding this comment

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

❤️

@nickfloyd nickfloyd self-requested a review January 7, 2026 16:16
@KyriosGN0 KyriosGN0 requested a review from nickfloyd January 8, 2026 06:46
@KyriosGN0
Copy link
Contributor Author

@nickfloyd could i get another look here?
Thanks!

@deiga
Copy link
Collaborator

deiga commented Jan 25, 2026

@KyriosGN0 while waiting could you rebase the branch?

@KyriosGN0
Copy link
Contributor Author

hey @nickfloyd, a gentle ping on this, Thanks!

@deiga deiga requested a review from stevehipwell February 8, 2026 09:58
Copy link
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

LGTM

@stevehipwell
Copy link
Collaborator

@KyriosGN0 could you please rebase this and then we can get it merged?

@stevehipwell stevehipwell added the Type: Documentation Improvements or additions to documentation label Feb 17, 2026
@stevehipwell stevehipwell added this to the v6.12.0 Release milestone Feb 17, 2026
…rganization_role_team resource

Signed-off-by: AvivGuiser <avivguiser@gmail.com>
Signed-off-by: AvivGuiser <avivguiser@gmail.com>
Signed-off-by: AvivGuiser <avivguiser@gmail.com>
@stevehipwell stevehipwell dismissed nickfloyd’s stale review February 17, 2026 10:15

The requested change was made.

@stevehipwell stevehipwell merged commit 46682cf into integrations:main Feb 17, 2026
7 checks passed
@KyriosGN0 KyriosGN0 deleted the sec-manager branch February 17, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOCS]: how to grant security admin role with role_team

5 participants