Conversation
- Rename skill path from inst/skills/r-package-maintainer to inst/skills/r-package-developer and update skill name metadata. - Move/retain all reference files under the new skill folder and add scripts/sync-from-github.sh in the renamed location. - Rename skills wrapper vignette to vignettes/skills-r-package-developer.qmd and remove the old maintainer wrapper. - Update pkgdown navigation/articles entries to point at skills-r-package-developer and refresh skills index wording. - Update skill policy/docs to remove automatic self-update requirement and keep manual update workflow explicit. - Add a user-facing README for the r-package-developer skill workflow and refine wording for Codex/Claude-style usage. - Refresh repository docs/metadata references (README/NEWS/PROJECT_DESIGN/.gitignore) to match the new skill naming and structure.
- Remove bundled r-package-developer skill files from inst/skills and stop treating it as package content. - Remove the pkgdown wrapper page for the external developer skill and drop it from pkgdown articles/navbar to avoid include/path breakage. - Update README and inst/skills index to point to external path skills/r-package-developer. - Add explicit policy text: if external skill is missing, suggest installation, but install only with explicit developer approval and never automatically. - Keep package-level skill index focused on bundled skills plus external-skill guidance. - Carry forward ignore/build-ignore adjustments for the new external skills layout.
- Bump package version in DESCRIPTION from 0.4.0 to 0.4.1. - Add NEWS entry for 0.4.1 documenting externalization of the r-package-developer skill and explicit install-approval policy. - Update PROJECT_DESIGN to reflect external developer-skill location (skills/r-package-developer) and align recent-change summary to 0.4.1. - Run full audit via devtools::check() with clean result: 0 errors, 0 warnings, 0 notes.
rkrug
changed the title
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.
Summary
r-package-developerfrom package content (inst/skills) to repository-levelskills/r-package-developer0.4.1and sync release metadata/docsChanges
inst/skills/r-package-developer/**vignettes/skills-r-package-developer.qmd_pkgdown.ymlto remove the external skill article entryREADME.mdandinst/skills/README.md:skills/r-package-developerDESCRIPTION->Version: 0.4.1NEWS.md-> new0.4.1entryPROJECT_DESIGN.md-> aligned to external skill location and 0.4.1 cycleValidation
devtools::check()