Skip to content

docs: Help Pane#73

Merged
isabelizimm merged 6 commits intomainfrom
docs-help
May 30, 2025
Merged

docs: Help Pane#73
isabelizimm merged 6 commits intomainfrom
docs-help

Conversation

@isabelizimm
Copy link
Contributor

@netlify
Copy link

netlify bot commented May 28, 2025

Deploy Preview for positron-posit-co ready!

Name Link
🔨 Latest commit ca0a631
🔍 Latest deploy log https://app.netlify.com/projects/positron-posit-co/deploys/68375d411c3ce100080d0b54
😎 Deploy Preview https://deploy-preview-73--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.

@netlify
Copy link

netlify bot commented May 28, 2025

Deploy Preview for positron-posit-co ready!

Name Link
🔨 Latest commit 4c4bd22
🔍 Latest deploy log https://app.netlify.com/projects/positron-posit-co/deploys/6839bcb59e454600090a13fc
😎 Deploy Preview https://deploy-preview-73--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.

Comment on lines +44 to +45
- Function and package documentation
- Package vignettes (when available)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is what is in the RStudio user guide, do we want to be more specific?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks good as is!

@isabelizimm isabelizimm marked this pull request as ready for review May 28, 2025 19:13
@isabelizimm isabelizimm requested a review from Copilot May 29, 2025 16:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds user documentation for the new Help Pane feature, integrates it into the site navigation, and provides Copilot with documentation review guidelines.

  • Introduced a help-pane.qmd page outlining how to use Help Pane in Python and R.
  • Updated _quarto.yml to include help-pane.qmd in the website menu.
  • Added .github/copilot-instructions.md with Copilot’s technical documentation review rules.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
help-pane.qmd New Help Pane guide with usage examples, supported content, and navigation tools
_quarto.yml Included help-pane.qmd in the website’s navigation order
.github/copilot-instructions.md Added Copilot prompt rules for reviewing technical docs
Comments suppressed due to low confidence (1)

help-pane.qmd:13

  • Inconsistent capitalization: ‘Help pane’ should be ‘Help Pane’ to match the component name throughout the document.
These methods will automatically open the Help pane and display documentation.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@isabelizimm
Copy link
Contributor Author

isabelizimm commented May 29, 2025

I added some Copilot instructions to help pre-review style and tone. We can iterate on the instructions, but it already had a good suggestion on this small doc addition!

It does make this PR feel a bit noisy before another human even sees it, but maybe we try it out and see if it feels like too much?

@isabelizimm isabelizimm requested a review from juliasilge May 29, 2025 16:12
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 so much!

Comment on lines +44 to +45
- Function and package documentation
- Package vignettes (when available)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks good as is!

Co-authored-by: Julia Silge <julia.silge@gmail.com>
@isabelizimm isabelizimm merged commit 70e14c5 into main May 30, 2025
5 checks passed
@isabelizimm isabelizimm deleted the docs-help branch May 30, 2025 14: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.

3 participants