Skip to content

Add dev container docs#283

Merged
jmcphers merged 7 commits intomainfrom
feature/devcontainer-docs
Feb 25, 2026
Merged

Add dev container docs#283
jmcphers merged 7 commits intomainfrom
feature/devcontainer-docs

Conversation

@jmcphers
Copy link
Contributor

Adds basic documentation for the Dev Containers feature.

@netlify
Copy link

netlify bot commented Feb 24, 2026

Deploy Preview for positron-posit-co ready!

Name Link
🔨 Latest commit 6bf3586
🔍 Latest deploy log https://app.netlify.com/projects/positron-posit-co/deploys/699e4a920cea2a000898b2c1
😎 Deploy Preview https://deploy-preview-283--positron-posit-co.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jmcphers jmcphers requested a review from juliasilge February 24, 2026 18:25
@juliasilge
Copy link
Collaborator

Can you try running the new Claude doc-reviewer skill on this PR?
https://github.com/posit-dev/doc-reviewer/

It should help you with alt text for images, using bold aligned with the style guide, etc. One thing I want to note is that commands should be italics, and pretty much the only things that should be bold are UI elements, like where you have "Dev Containers".

@juliasilge
Copy link
Collaborator

You might get pretty far by asking Claude to use the CLAUDE.md in this repo too.

@jmcphers
Copy link
Contributor Author

@juliasilge done!

Copy link
Collaborator

@juliasilge juliasilge left a comment

Choose a reason for hiding this comment

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

Thank you!

I apologize so much for not being clear enough before, but the images do need alt text and not captions:
https://github.com/posit-dev/doc-reviewer/blob/main/skills/doc-reviewer/posit-style-guide.md

To get more explicit, that is like this:

![Action Bars](images/action-bars.png){fig-alt="Positron IDE showing action bars in various panes including the Editor, Console, Variables pane, and Plots pane."}

The captions are typically shorter and meant for any reader, while the alt text is longer and meant for folks with vision differences.

Did the skill or the prompting not correctly identify that it's alt text we need? Maybe we can be more strong about that somewhere.

@jmcphers
Copy link
Contributor Author

Did the skill or the prompting not correctly identify that it's alt text we need? Maybe we can be more strong about that somewhere.

Yeah, the skill (I used doc-reviewer) guided me to make the caption into a full sentence and didn't prompt me to add alt text at all. Probably could use an update!

jmcphers and others added 2 commits February 24, 2026 16:46
Co-authored-by: Julia Silge <julia.silge@gmail.com>
Copy link
Collaborator

@juliasilge juliasilge left a comment

Choose a reason for hiding this comment

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

Thank you! 🐳

@jmcphers jmcphers merged commit a6980e6 into main Feb 25, 2026
5 checks passed
@jmcphers jmcphers deleted the feature/devcontainer-docs branch February 25, 2026 01:24
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