Apply new Claude doc-reviewer skill to whole site#282
Merged
juliasilge merged 7 commits intomainfrom Feb 26, 2026
Merged
Conversation
✅ Deploy Preview for positron-posit-co ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
sharon-wang
previously approved these changes
Feb 25, 2026
Member
sharon-wang
left a comment
sharon-wang
left a comment
There was a problem hiding this comment.
Thank you for doing this sweep!
Co-authored-by: sharon <sharon-wang@users.noreply.github.com>
Collaborator
AshleyHenry15
left a comment
AshleyHenry15
left a comment
There was a problem hiding this comment.
It did ok. There was a blanket rule to switch all clicks to select but we have updated that rule and it will change the next time we release. Although this looks pretty good, do you want to wait until we release a new version so all of the clicks aren't updating to select?
assistant-chat-context.qmd
Outdated
| To include additional attachments as context for your prompts: | ||
|
|
||
| - Click the {{< fa paperclip >}} paperclip button in the chat pane | ||
| - Select the {{< fa paperclip >}} paperclip button in the chat pane |
Collaborator
There was a problem hiding this comment.
We've recently updated the doc-reviewer skill to ignore trying to switch all "clicks" to "select" so a new version of the skill would ignore this. I think we can ignore this and the previous suggestion if you want.
assistant-chat-context.qmd
Outdated
| ### Toggle the visibility of the active interpreter session | ||
|
|
||
| In the chat pane, click the {{< fa eye >}} eye button on the interpreter session attachment to toggle whether Assistant can access the session data. | ||
| In the chat pane, select the {{< fa eye >}} eye button on the interpreter session attachment to toggle whether Assistant can access the session data. |
Collaborator
There was a problem hiding this comment.
Same comment as above.I can release a new version this week
assistant-chat-context.qmd
Outdated
| ### Manually attach an interpreter session | ||
|
|
||
| 1. Click the {{< fa paperclip >}} paperclip button in the chat pane | ||
| 1. Select the {{< fa paperclip >}} paperclip button in the chat pane |
assistant-chat-context.qmd
Outdated
| 1. Open the Source Control pane (_Source Control: Focus on Changes View_ in the Command Palette). | ||
| 1. Ensure you have staged changes to commit. | ||
| 1. Click the sparkles icon in the commit message input box. | ||
| 1. Select the sparkles icon in the commit message input box. |
assistant-chat.qmd
Outdated
| ### Opening the chat pane | ||
|
|
||
| To open the chat pane, click the chat robot icon in the sidebar or run the command _Chat: Open Chat_ from the Command Palette. | ||
| To open the chat pane, select the chat robot icon in the sidebar or run the command _Chat: Open Chat_ from the Command Palette. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR applies our new
doc-reviewerskill to the site as a whole:https://github.com/posit-dev/doc-reviewer/
@AshleyHenry15 would you be willing to take a look at this and spot check that we are applying this as intended? I don't think it needs an extremely deep review, but we would so appreciate you taking a look to see if anything is not what you would have expected.