Skip to content

Use rdtools#337

Open
hadley wants to merge 9 commits into
mainfrom
use-rdtools
Open

Use rdtools#337
hadley wants to merge 9 commits into
mainfrom
use-rdtools

Conversation

@hadley

@hadley hadley commented Jul 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@hadley hadley requested a review from lionel- July 7, 2026 12:41
Comment thread NEWS.md

* `dev_example()` now expands the package's custom Rd macros when running examples, matching the behaviour of `dev_help()`.
* `dev_help()` and friends now use the rdtools package to build and look up the topic index, which is considerably faster. `dev_topic_index()` now returns a named character vector mapping alias to Rd file name (without extension), and an alias documented in multiple Rd files now triggers a warning.
* `dev_help()` no longer carries a workaround for versions of RStudio that predate `previewRd()`; outside of RStudio, HTML help now opens directly in your browser.

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.

Maybe mention the year? I saw in removed code that the minimum version is from 2022?

Comment thread R/dev-help.R
rstudio_preview <- type == "html" &&
is_installed("rstudioapi") &&
rstudioapi::isAvailable() &&
rstudioapi::hasFun("previewRd")

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.

Maybe keep the "2022.12.0.256" check and ref to rstudio#12111?

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