fix: correct devcontainer config structure in docs (#2068)#2074
Conversation
- Fix blog post and command docs that incorrectly nested devcontainer under components: devcontainer is a top-level atmos.yaml key, not a component concept - Clarify that both inline YAML config and devcontainer.json imports are supported - Embed working example in blog post for better discoverability - Update 8 YAML code blocks across 2 files to show correct structure Users were confused by outdated docs showing wrong config structure, taking an hour to debug (.json vs atmos.yaml). Both are now supported and properly documented. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Dependency Review✅ No vulnerabilities or license issues found.Scanned FilesNone |
📝 WalkthroughWalkthroughMoves devcontainer configuration examples from a nested Changes
Sequence Diagram(s)(omitted) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 🧹 Recent nitpick comments
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2074 +/- ##
==========================================
+ Coverage 76.30% 76.32% +0.01%
==========================================
Files 802 802
Lines 75521 75521
==========================================
+ Hits 57630 57639 +9
+ Misses 14302 14296 -6
+ Partials 3589 3586 -3
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add cloudposse-corp to lychee exclude list since it's a private org that returns 404 for unauthenticated CI runners. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
These changes were released in v1.206.2. |
|
These changes were released in v1.207.0. |
|
These changes were released in v1.208.0-rc.0. |
|
These changes were released in v1.208.0-test.15. |
what
atmos.yamlkey, not nested undercomponents:devcontainer.jsonimports are supportedwhy
Users were unable to follow the blog post instructions and spent significant time debugging (issue #2068). The outdated docs incorrectly nested devcontainer configuration under
components:, conflicting with the actual schema and causing confusion about whether to use.jsonoratmos.yamlfiles. Both are now properly supported and documented.references
Closes #2068
Summary by CodeRabbit
Documentation
Chores