Skip to content

Add a built-in for setting the PPM repo in a running session#980

Merged
jmcphers merged 1 commit intomainfrom
aj-set-ppm-repo-builtin
Feb 2, 2026
Merged

Add a built-in for setting the PPM repo in a running session#980
jmcphers merged 1 commit intomainfrom
aj-set-ppm-repo-builtin

Conversation

@atheriel
Copy link
Contributor

@atheriel atheriel commented Dec 5, 2025

This commit adds a .ps.set_ppm_repo() built-in that works the same way as the existing CLI option, but allows the repo to be set at runtime.

It is intended to support making the positron.r.packageManagerRepository setting apply without needing to restart/recreate existing R sessions.

Part of posit-dev/positron#10965.

Copy link
Contributor

@lionel- lionel- left a comment

Choose a reason for hiding this comment

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

LGTM

This commit adds a `.ps.set_ppm_repo()` built-in that works the same way
as the existing CLI option, but allows the repo to be set at runtime.

It is intended to support making the
`positron.r.packageManagerRepository` setting apply without needing to
restart/recreate existing R sessions.

Part of posit-dev/positron#10965.

Signed-off-by: Aaron Jacobs <aaron.jacobs@posit.co>
@atheriel atheriel force-pushed the aj-set-ppm-repo-builtin branch from 0cf3a74 to ffb4007 Compare January 6, 2026 18:03
@lionel-
Copy link
Contributor

lionel- commented Jan 27, 2026

Hi @atheriel, did you want us to merge the PR? We usually just approve and let the author merge if they have write rights.

@jmcphers
Copy link
Contributor

jmcphers commented Feb 2, 2026

@atheriel confirmed this is OK to merge.

@jmcphers jmcphers merged commit f6c1ba3 into main Feb 2, 2026
14 of 15 checks passed
@jmcphers jmcphers deleted the aj-set-ppm-repo-builtin branch February 2, 2026 17:20
@github-actions github-actions bot locked and limited conversation to collaborators Feb 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants