Skip to content

Switch to 'asis' vignette builder from Rcpp 1.1.0.1#80

Merged
eddelbuettel merged 1 commit intomasterfrom
feature/vignette_rework
Jul 20, 2025
Merged

Switch to 'asis' vignette builder from Rcpp 1.1.0.1#80
eddelbuettel merged 1 commit intomasterfrom
feature/vignette_rework

Conversation

@eddelbuettel
Copy link
Owner

No description provided.

@eddelbuettel eddelbuettel requested a review from Copilot July 20, 2025 14:08
Copy link

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 switches the vignette builder from a LaTeX-based approach to using the 'asis' vignette builder introduced in Rcpp 1.1.0.1. This change simplifies the vignette handling by removing LaTeX boilerplate code and leveraging Rcpp's new vignette building capabilities.

  • Migrates vignette to use Rcpp's 'asis' builder instead of LaTeX wrapper
  • Updates package dependencies to require Rcpp >= 1.1.0.1
  • Temporarily installs development version of Rcpp in CI to access new functionality

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
vignettes/UsingAnnoyInCpp.pdf.asis Removes LaTeX boilerplate and adds Rcpp asis vignette engine metadata
DESCRIPTION Bumps version, updates Rcpp dependency requirement, and adds VignetteBuilder field
ChangeLog Documents the changes made in this release
.github/workflows/ci.yaml Adds temporary installation of Rcpp development version for CI

@eddelbuettel eddelbuettel force-pushed the feature/vignette_rework branch from 9f4b252 to 06f07e9 Compare July 20, 2025 14:11
@eddelbuettel eddelbuettel force-pushed the feature/vignette_rework branch from 06f07e9 to d63c071 Compare July 20, 2025 14:11
@eddelbuettel
Copy link
Owner Author

eddelbuettel commented Jul 20, 2025

If a CRAN release is needed or made between now and the release of Rcpp 1.1.1 this PR will have to be withdrawn first.

(Or the vignette has to be withdrawn, or the vignette builder has to be R.rsp for the interim.)

@eddelbuettel eddelbuettel merged commit 11b8aba into master Jul 20, 2025
2 checks passed
@eddelbuettel eddelbuettel deleted the feature/vignette_rework branch July 20, 2025 14:35
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