chore: finish the extendr.rs URL move (README, pkgdown) - #11
Merged
Conversation
Both https://extendr.github.io/ and https://extendr.github.io/extendr/extendr_api/ 301-redirect to extendr.rs. README.md is not in .Rbuildignore, so it ships in the tarball and R CMD check --as-cran reports both under "CRAN incoming feasibility". Hand-edited rather than re-rendered from README.qmd so the embedded example output dates are not restamped.
The site is served from extendr.rs via the org's custom domain, so this is what pkgdown should use for canonical link tags and sitemap.xml. Also brings _pkgdown.yml back in line with the URL field in DESCRIPTION.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TODO: replace this line with your own framing, in your own voice.
AI-written details
Summary
Finishes the
extendr.github.iotoextendr.rsmove that #10 started onDESCRIPTION. Two commits, deliberately separate because they carry different risk:d39bac9README (README.md:7,13and theREADME.qmd:9,14mirrors). Clears a realR CMD check --as-cranNOTE.2d57150pkgdown (_pkgdown.yml:1). Cosmetic on its own, but changes canonical-link andsitemap.xmloutput on the live site.Drop either commit without unpicking the other if you only want one.
Why the README change matters
README.mdis not in.Rbuildignore, so it ships in the tarball and CRAN attributes its links to tomledit. Both old URLs 301-redirect, andR CMD check --as-cranon currentmainreports:This is pre-existing on
main, not a regression from #10.README.mdwas hand-edited rather than re-rendered fromREADME.qmd. A re-render would restamp the embedded example output dates (7fb9808 shows that happening:date = 2025-02-20became2025-03-03), which would bury a 2-line URL fix in unrelated churn.Verification (local, R 4.6.0)
R CMD check --as-cranon the built tarball is down toStatus: 2 NOTEs, with no invalid-URL findings at all. What remains is unrelated to this PR:CRAN incoming feasibilitynow contains onlyVersion contains large components (0.1.1.9000), expected for a dev version.checking for hidden files and directoriespicks up a local.claude/directory in my working tree. It does not exist in a fresh checkout, so CI will not see it.urlchecker::url_check(".")goes from 2 findings to 0. Scope note: urlchecker readsDESCRIPTION,man/,README.mdandNEWS.md, so that 0 validates theREADME.mdlines specifically. TheREADME.qmdedit is source-sync only (it is.Rbuildignored and never checked), and_pkgdown.ymlis not in urlchecker's scope either. It parses cleanly tourl: https://extendr.rs/tomledit/and thepkgdownworkflow job is the real check on it.Noticed but deliberately not changed
The extendr badge label reads
extendr-^0.7.1whilesrc/rust/Cargo.toml:13pinsextendr-api = "0.8.2". Only the badge's link target was touched here, not the version it asserts, since whether that should track Cargo.toml or advertise a supported floor is your call. Happy to bump it if you want.Drafted by Claude (claude-opus-5). Reviewed by the author.