Skip to content

Comments

Simplify sandbox setup documentation#574

Merged
michaeljguarino merged 1 commit intomainfrom
simplify-sandbox-documentation
Feb 19, 2026
Merged

Simplify sandbox setup documentation#574
michaeljguarino merged 1 commit intomainfrom
simplify-sandbox-documentation

Conversation

@michaeljguarino
Copy link
Member

This consolidates the guidance into one helm values file example

Plural Flow: docs
Plural Preview: docs

This consolidates the guidance into one helm values file example
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-574 is building...

Name Details
⚡ Service docs-pr-574 (4 / 5 ready)
☁️ Cluster plural (eks)
:octocat: Commit sha 82f6225
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 19, 2026

Greptile Summary

This PR consolidates sandbox setup documentation by adding a TLDR section with a comprehensive Helm values example that combines all the configuration settings previously scattered throughout the document.

  • Added a new TLDR section (lines 22-83) providing a single, consolidated Helm values file example
  • The example covers airgap configuration, custom OIDC, license keys, registry settings, KAS setup, and git server configuration
  • Improves developer experience by providing a quick-start reference before diving into detailed explanations
  • Found one typo: enteprise instead of enterprise on line 68

Confidence Score: 4/5

  • This PR is safe to merge with minimal risk - it's a documentation-only change that improves usability
  • Documentation improvements with clear benefit to users. Only issue is a minor typo that should be fixed before merging to maintain professionalism
  • No files require special attention beyond fixing the typo in the YAML example

Important Files Changed

Filename Overview
pages/getting-started/advanced-config/sandboxing.md Added TLDR section with consolidated Helm values example; found one typo in configuration

Last reviewed commit: 82f6225

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.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

agent:
proxy:
image:
repository: your.enteprise.registry/some/nginx
Copy link

Choose a reason for hiding this comment

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

Typo: enteprise should be enterprise

Suggested change
repository: your.enteprise.registry/some/nginx
repository: your.enterprise.registry/some/nginx

@michaeljguarino michaeljguarino merged commit de3c2d8 into main Feb 19, 2026
13 checks passed
@michaeljguarino michaeljguarino deleted the simplify-sandbox-documentation branch February 19, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant