Skip to content

Comments

Add docs for PR governance controllers#577

Merged
michaeljguarino merged 1 commit intomainfrom
pr-governance-docs
Feb 23, 2026
Merged

Add docs for PR governance controllers#577
michaeljguarino merged 1 commit intomainfrom
pr-governance-docs

Conversation

@michaeljguarino
Copy link
Member

Adds documentation for:

  1. webhook governance
  2. servicenow governance

Plural Flow: docs
Plural Preview: docs

@michaeljguarino michaeljguarino requested a review from a team as a code owner February 22, 2026 21:14
@michaeljguarino michaeljguarino added new-content For new docs content. enhancement labels Feb 22, 2026
Copy link
Contributor

@plural-copilot plural-copilot bot left a comment

Choose a reason for hiding this comment

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

✅ Deploy preview for docs-pr-577 is ready!

Name Details
⚡ Service docs-pr-577 (5 / 5 ready)
☁️ Cluster plural (eks)
:octocat: Commit sha a16a45a
Plural URL https://console.mgmt.plural.sh/cd/clusters/446acdef-8524-425a-a49f-5336a...
🔍 Logs URL https://console.mgmt.plural.sh/cd/clusters/446acdef-8524-425a-a49f-5336a...
🌊 Flow URL https://console.mgmt.plural.sh/flows/08a17af1-8578-449e-9ae1-28ccd97eafc...
Preview Link Preview

You can learn more about preview environments and Plural Flows here

@greptile-apps
Copy link

greptile-apps bot commented Feb 22, 2026

Greptile Summary

Adds comprehensive documentation for PR governance controllers, including ServiceNow integration and custom webhook support. The PR also documents PR description patterns for correlating pull requests with Plural resources.

Key additions:

  • Documentation for webhook-based and ServiceNow governance controllers
  • PR description pattern reference guide covering service/stack correlation, merge scheduling, and preview environments
  • Enhanced preview environment docs with namespace cleanup configuration

One syntax error found that will cause the mermaid diagram to fail rendering in webhook.md.

Confidence Score: 4/5

  • Safe to merge after fixing the mermaid syntax error on line 82 of webhook.md
  • Documentation changes are low-risk, but the unclosed mermaid block in webhook.md will cause rendering issues. Previous review comments identified several typos that should be addressed.
  • Pay attention to pages/plural-features/pr-automation/governance/webhook.md for the mermaid syntax fix

Important Files Changed

Filename Overview
pages/plural-features/pr-automation/description-patterns.md New documentation page explaining PR description patterns for correlating PRs with Plural resources
pages/plural-features/pr-automation/governance/index.md Overview documentation for PR governance controllers with CRD examples
pages/plural-features/pr-automation/governance/servicenow.md ServiceNow integration documentation with change request synchronization workflow
pages/plural-features/pr-automation/governance/webhook.md Custom webhook governance documentation with mermaid syntax error on line 82

Last reviewed commit: ffff214

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

6 files reviewed, 11 comments

Edit Code Review Agent Settings | Greptile

@michaeljguarino michaeljguarino force-pushed the pr-governance-docs branch 3 times, most recently from 0a6caf9 to ffff214 Compare February 22, 2026 21:35
@michaeljguarino
Copy link
Member Author

@greptileai

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

7 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@michaeljguarino michaeljguarino force-pushed the pr-governance-docs branch 4 times, most recently from 8a38a4e to d11b55c Compare February 22, 2026 23:55
Adds documentation for:

1. webhook governance
2. servicenow governance
@michaeljguarino michaeljguarino merged commit 27466ac into main Feb 23, 2026
11 checks passed
@michaeljguarino michaeljguarino deleted the pr-governance-docs branch February 23, 2026 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement new-content For new docs content.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants