Skip to content

feat: some tips based on New Goblins experience with the platform.#818

Merged
jburns24 merged 2 commits intomasterfrom
colin/platform-rework
Apr 14, 2026
Merged

feat: some tips based on New Goblins experience with the platform.#818
jburns24 merged 2 commits intomasterfrom
colin/platform-rework

Conversation

@Valinora
Copy link
Copy Markdown
Contributor

@Valinora Valinora commented Jan 5, 2026

Just a couple of tips.

After #814 is merged, will do another pass.

Summary by CodeRabbit

  • Documentation
    • Clarified guidance on deploy repository workflow setup and remote state configuration.
    • Added considerations for GitOps approach with ArgoCD installation.
    • Improved formatting in exercise instructions.

@jburns24 jburns24 marked this pull request as ready for review April 14, 2026 14:10
@jburns24 jburns24 merged commit ddb2b82 into master Apr 14, 2026
2 checks passed
@jburns24 jburns24 deleted the colin/platform-rework branch April 14, 2026 14:10
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 14, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4d47f770-9cc6-433c-85e6-015f93f1852c

📥 Commits

Reviewing files that changed from the base of the PR and between d3849dd and 77cab3b.

📒 Files selected for processing (1)
  • docs/10-platform-engineering/10.2-platforms.md

📝 Walkthrough

Walkthrough

Documentation updates to the platform engineering guide clarifying TVP repo-setup instructions for remote state configuration, adding a Terragrunt workflow checklist item, refining ArgoCD introduction language, including a new consideration note on GitOps approaches, and adjusting Exercise 4 formatting.

Changes

Cohort / File(s) Summary
Documentation Updates
docs/10-platform-engineering/10.2-platforms.md
Added guidance on remote state creation in TVP repo-setup section, included deploy-repo Terragrunt workflow checklist item, refined ArgoCD introduction wording, added "Something to consider" admonition questioning long-term Terraform-based ArgoCD installation viability, adjusted Exercise 4 indentation, and removed "UNCHARTED TERRITORY" heading.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A document refined with care and thought,
Remote state guidance, workflows taught,
ArgoCD wisdom in admonition's glow,
Indentation fixed for better flow,
Platform engineering guides us true! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch colin/platform-rework

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

2 participants