Skip to content

Positron update - #8

Merged
awurno merged 9 commits into
mainfrom
positron-update
Sep 9, 2026
Merged

Positron update#8
awurno merged 9 commits into
mainfrom
positron-update

Conversation

@awurno

@awurno awurno commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Don't look at the branch name

@awurno
awurno requested a review from GreatEmerald August 27, 2026 12:25

@GreatEmerald GreatEmerald left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hmm, all the files in _site are a bit messy, indeed. I assume that's what could be put into gh-pages and the main branch can be kept clean? Also some other comments below.

Comment thread index.qmd Outdated
Comment thread index.qmd Outdated

Both the `R` and the `python` interpreters can be run from Bash. Here is an example of code that is correct in both Python and R. You can use this example to execute a script written in either language:
```{bash}
```{bash, eval=FALSE}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why all the eval=FALSE? We do want to test Bash and Python on the VMs and also get the output in the HTML.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do we want the results in the HTML? I prefer them having to copy and paste and see for them selves. I changed because somewhere next there is a sudo command and rendering complained, so I found/replaced all.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, we do want to show the output, because the students should be able to verify that what they are doing is correct. Some are using their own Linux versions etc.

awurno and others added 3 commits August 31, 2026 11:36
Co-authored-by: Dainius Masiliūnas <dainius.masiliunas@wur.nl>

@GreatEmerald GreatEmerald left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See one issue below.

Comment thread index.html Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Entirely removing the HTML will also be a problem, though, because we will get a 404 on the website if this is merged. If you add a runner, it has to go into .github/actions/something.yml from what I know.

Comment thread index.qmd Outdated
theme: simplex
highlight-style: zenburn
toc: true
toc-location: left

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why left? Shouldn't that be right to align better with the RMarkdown stuff?

@awurno
awurno merged commit 4455dbf into main Sep 9, 2026
@awurno
awurno deleted the positron-update branch September 9, 2026 12:32
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