Conversation
New reference and troubleshooting page covering required permissions, event subscriptions, and steps to update the CircleCI GitHub App configuration on GHES. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add overview section with context on App creation and ownership - Move verification instructions higher in the page - Add event subscriptions and permissions warning screenshots - Add note about other permissions, convert explanations to bullet lists - Remove GHES acronym from file name, nav, and content - Apply style guide fixes (active voice, sentence length, terminology) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
BeFunes
force-pushed
the
ghes-github-app-configuration
branch
from
db28d15 to
1671358
Compare
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
There are currently two linting errors, because the linter wants to add full stops at the end of the events in the list I don't love this - I'll hold for @rosieyohannan to recommend the best path forward |
| If your App configuration does not match the requirements above, you may see a warning in CircleCI on your GitHub App Server card: **"Some CircleCI features require updated App permissions."** | ||
|
|
||
| .Warning displayed when App permissions need updating | ||
| image::guides:ROOT:ghes-app-permissions-warning.png[CircleCI warning banner showing that some features require updated App permissions with a Learn how to fix this link,width=400] |
There was a problem hiding this comment.
I tried to fix the width to 400px, but it didn't work. @rosieyohannan
There was a problem hiding this comment.
Currently we have CSS overriding individual image scaling. We do not have any images on the site that aren't full width. If we want to be able to scale images I can add this ability but would like to explore whether we actually want it first
| * Pull request review | ||
| * Pull request review comment |
There was a problem hiding this comment.
The linter wants this:
| * Pull request review | |
| * Pull request review comment | |
| * Pull request review. | |
| * Pull request review comment. |
There was a problem hiding this comment.
Yeah agreed this rule is too strict. Currently Its set to flag a full stop when the list item is more than two words. We use a full stop when a list item is a full sentence or the end of a sentence. Just ignore for now and I will try to refine the rule
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
.../guides/modules/integration/pages/github-app-configuration-for-github-enterprise-server.adoc
Show resolved
Hide resolved
2 participants
Summary
Test plan
guides/integration/ghes-github-app-configuration/🤖 Generated with Claude Code