Skip to content

Conversation

@alanjclark
Copy link

@alanjclark alanjclark commented Apr 16, 2025

Description

Clarifies that Restricted Contexts is available on GH App pipelines on GH Oauth connected orgs.

Reasons

Wording was implying that this was only available on GH OAuth

Content Checklist

Please follow our style when contributing to CircleCI docs. Our style guide is here: https://circleci.com/docs/style/style-guide-overview.

Please take a moment to check through the following items when submitting your PR (this is just a guide so will not be relevant for all PRs) 😸:

  • Break up walls of text by adding paragraph breaks.
  • Consider if the content could benefit from more structure, such as lists or tables, to make it easier to consume.
  • Keep the title between 20 and 70 characters.
  • Consider whether the content would benefit from more subsections (h2-h6 headings) to make it easier to consume.
  • Check all headings h1-h6 are in sentence case (only first letter is capitalized).
  • Is there a "Next steps" section at the end of the page giving the reader a clear path to what to read next?
  • Include relevant backlinks to other CircleCI docs/pages.

Rewords the requirements for Restricted Contexts to confirm that we now support GH App on GH Oauth connected orgs (hybrid).
@alanjclark alanjclark requested review from a team as code owners April 16, 2025 16:07
@sebastian-lerner sebastian-lerner self-requested a review April 16, 2025 16:11
@alanjclark alanjclark requested a review from gordonsyme April 16, 2025 16:17
Updates spelling of Organization (British problems) and adds exception for custom webhooks.
@alanjclark
Copy link
Author

Updated with review recommendations, and also added exception for custom webhooks as per Slack conversation.

Switches wording to clarify this is organizations rather than user accounts.
@gordonsyme gordonsyme dismissed sebastian-lerner’s stale review May 7, 2025 13:54

Sebastian is on PTO

alanjclark added 2 commits May 8, 2025 11:12
Removes the anchor from the GitHub integration URL as I suspect the tip I was linking to has now moved to the top of the doc.
Adds hash missing from link
@alanjclark alanjclark merged commit 6049917 into circleci:master May 8, 2025
2 of 3 checks passed
@alanjclark alanjclark deleted the restricted-contexts-gh-app branch May 8, 2025 10:29
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.

4 participants