Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change code blocks to not run code #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Change code blocks to not run code #51

wants to merge 1 commit into from

Conversation

wch
Copy link
Contributor

@wch wch commented Dec 19, 2023

I noticed that when building the docs, a couple of pages used Jupyter to run R and Python code, even though it wasn't actually necessary to run code.

This PR changes it so that the it doesn't have to run any code, but keeps the highlighting. This makes the build time a bit faster.

@wch wch requested a review from gshotwell December 19, 2023 18:40
Comment on lines -55 to -56
```{python}
#| source-line-numbers: "9-13"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I noticed that this highlighting wasn't actually used because the line-highlight extension isn't enabled for this page, and when I added the extension, it highlighted what was probably the wrong set of lines.

But if line highlighting is desirable here, we could change change things to enable the extension and fix the highlighted lines.

This is what it currently looks like on the web site:

image

https://shiny.posit.co/py/docs/workflow-modules.html

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.

None yet

1 participant