ci(citation): call the org reusable citation workflow - #370
Merged
Conversation
Replaces the hand-rolled "Update CITATION.cff" job with a thin caller. No overrides -- every default in the template suits this package. Four repos carried near-identical copies of this job, and two things had gone per-repo that should not have been: - the install-spatial-deps pin, which had drifted across the four to @v0.1, @v0.2 and a raw SHA. Every tag v0.1-v0.5 still adds the ubuntugis-unstable PPA (libgdal37), ABI-incompatible with the binaries in Posit's noble cache. The template reaches the action through setup-r-deps, so the pin is internal to PredictiveEcology/actions and moves once. - the privilege split: generating CITATION.cff installs and runs third-party R code, and committing it needs a write-scoped token. The template runs those as separate jobs. Until now only SpaDES.tools did. Also drops `master` from the push trigger -- this repo has no master branch -- and the pandoc step, which nothing in this job uses. Requires PredictiveEcology/actions#35. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015ZoicL7829pkb5ZbS5Ciww
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.
Replaces the hand-rolled
Update CITATION.cffjob with a thin caller. No overrides — every default in the template suits this package.Overlaps #369, which also rewrites this file on
development. Whichever merges second will need a rebase. This branch also drops theNLMR/fastshppost-install step: neither is a dependency of SpaDES (its Suggests are covr, knitr, rmarkdown, testthat), and thefastshpsource build is what failed this workflow on 2026-08-29.Why template this one
Four repos (
reproducible,SpaDES,SpaDES.core,SpaDES.tools) carried near-identical copies of this job, and two things had gone per-repo that shouldn't have:install-spatial-depspin. A hand-rolled caller names that action directly, so the four had drifted to@v0.1,@v0.2and a raw SHA. Every tagv0.1–v0.5still adds theubuntugis-unstablePPA (libgdal37), which is ABI-incompatible with the binaries in Posit's noble cache. The template reaches the action throughsetup-r-deps, so the pin is internal toPredictiveEcology/actionsand moves once instead of in four PRs.CITATION.cffinstalls and runs third-party R code; committing it needs a write-scoped token. The template runs those as separate jobs —buildwithcontents: read,commitwithcontents: writeand nothing but git. Until now onlySpaDES.toolsdid this.Also dropped
masterfrom the push trigger — this repo has nomasterbranch.setup-pandocstep where present.cff_write()readsDESCRIPTIONandinst/CITATIONand writes YAML; nothing renders.Behaviour
Unchanged. The template still runs with
dependencies: "all", socff_write(dependencies = TRUE)sees the same installed set and the generatedreferences:list comes out the same.Ordering
Blocked on PredictiveEcology/actions#35, which adds the template. This PR will fail until that merges.
🤖 Generated with Claude Code
https://claude.ai/code/session_015ZoicL7829pkb5ZbS5Ciww