Skip to content

Fix R-devel DESCRIPTION compatibility and CRAN detection - #115

Merged
dieghernan merged 2 commits into
mainfrom
codex/r-devel-description-encoding
Oct 5, 2026
Merged

dieghernan merged 2 commits into
mainfrom
codex/r-devel-description-encoding

Conversation

@dieghernan

@dieghernan dieghernan commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Description

R-devel reads DCF files as UTF-8 and escapes invalid bytes instead of converting them using the Encoding field. This caused the Latin-1 surveillance fixture to lose Michael Höhle's name. Newly enabled snapshots also exposed a pre-existing CRAN detection problem: CI used cloud.R-project.org while the test catalogue used cloud.r-project.org.

  • normalize legacy DESCRIPTION input before parsing, preserve CITATION encoding and leave source files unchanged
  • normalize repository domain case and trailing slashes, respect host and path boundaries and handle duplicate catalogue entries
  • align the simulated test catalogue with its configured repository
  • run version-dependent snapshots on R >= 4.6.0 and add regression tests
  • document exported as.cff(x) usage for R-devel checks
  • update README usage figures and CodeMeta file size

Related issue

No linked issue. See the R-devel DCF encoding changes.

Testing

  • complete devtools::test() suite passed on R 4.6.1 and R 4.7.0-devel before the CRAN follow-up, with a valid UTF-8 locale and TESTTHAT_PARALLEL=FALSE
  • CRAN regression tests passed on R 4.6.1 and R 4.7.0-devel; affected creation and merging tests passed with CI's repository configuration
  • existing snapshots match without updates
  • R-devel documentation checks and pkgdown::check_pkgdown() passed
  • devtools::lint() reported no lints; jarl check . passed

Checklist

  • I have added tests for user-facing changes.
  • I have added a bullet to NEWS.md for user-facing changes.
  • I have updated generated documentation after changing roxygen2 comments.

Normalize legacy DCF input before parsing, preserve citation encoding and run snapshots on R >= 4.6.0.
@dieghernan dieghernan added bug an unexpected problem or unintended behavior upkeep maintenance, infrastructure, and similar ai-assisted 🤖 changes include AI-assisted development or review labels Oct 5, 2026
@coveralls

coveralls commented Oct 5, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 37319170915

Coverage remained the same at 100.0%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 33 of 33 lines across 3 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 2371
Covered Lines: 2371
Line Coverage: 100.0%
Coverage Strength: 3132.95 hits per line

💛 - Coveralls

Normalize repository URLs, align the test repository fixture and include README and package metadata updates.
@dieghernan dieghernan changed the title Preserve legacy DESCRIPTION encoding with R-devel Fix R-devel DESCRIPTION compatibility and CRAN detection Oct 5, 2026
@dieghernan
dieghernan merged commit a2dc643 into main Oct 5, 2026
14 checks passed
@dieghernan
dieghernan deleted the codex/r-devel-description-encoding branch October 5, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assisted 🤖 changes include AI-assisted development or review bug an unexpected problem or unintended behavior upkeep maintenance, infrastructure, and similar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants