chore: point DESCRIPTION URL at extendr.rs - #10
Merged
Conversation
The extendr org moved its documentation to the extendr.rs custom domain, so https://extendr.github.io/tomledit/ now 301-redirects to https://extendr.rs/tomledit/. Use the canonical URL, matching rextendr's DESCRIPTION, and update the roxygen-generated Rd to agree. Closes #3
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
1 participant
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
DESCRIPTION:URL:now useshttps://extendr.rs/tomledit/instead ofhttps://extendr.github.io/tomledit/.man/tomledit-package.Rd: updated the matching\url{}in the "Useful links" block (roxygen output from theURL:field), hand-edited soRoxygenNote:and unrelated.Rdfiles stay untouched.Why this diff does not match the issue text
Issue #3 (Feb 2025) asked to change
https://josiahparry.github.io/tomledit/, and that specific change already landed in 7fb9808. The URL has since gone stale a second time: the extendr org moved its docs to theextendr.rscustom domain, so the github.io address is no longer canonical.https://extendr.github.io/tomledit/returns301 Moved Permanentlytohttps://extendr.rs/tomledit/.gh api repos/extendr/tomledit/pagesreports"html_url": "https://extendr.rs/tomledit/".rextendr'sDESCRIPTIONalready useshttps://extendr.rs/rextendr/, so this matches the sibling package.Verification (run locally, R 4.6.0)
urlchecker::url_check("."), before and after:mainextendr.github.io/tomledit/(DESCRIPTION, man/tomledit-package.Rd) + 2 in README.mdR CMD check --as-cranon the built tarball:checking Rust compilation ... OK,checking tests ... OK,checking Rd cross-references ... OK. TheCRAN incoming feasibilityNOTE no longer mentions the tomledit docs URL. Remaining ERROR/WARNING in that run were a local LaTeX gap (inconsolata.stynot found) and not package defects.Re-running
roxygen2::roxygenise()produced no diff in the\url{}block, confirming the hand-edited.Rdmatches generated output.Notes
README.md:7andREADME.md:13still link tohttps://extendr.github.io/andhttps://extendr.github.io/extendr/extendr_api/, both of which 301 toextendr.rs. BecauseREADME.mdis not in.Rbuildignoreit ships in the tarball, soR CMD check --as-cranattributes those to tomledit and NOTEs them. This is pre-existing onmain, not introduced here, and is left out to keep this PR scoped to chore: change url in description #3. Fixing those two (plus theirREADME.qmdmirrors, hand-edited so re-rendering does not restamp the embedded example dates) would take the count to 0. Happy to fold it in or send it separately, whichever you prefer._pkgdown.yml:1still hasurl: https://extendr.github.io/tomledit/. Left alone deliberately: it changes canonical-link and sitemap output on a live site, andrextendrcurrently has the same DESCRIPTION/_pkgdown.ymlsplit without build problems.Drafted by Claude (claude-opus-5). Reviewed by the author.