Skip to content

ci: pin PredictiveEcology/actions to @main - #30

Closed
eliotmcintire wants to merge 1 commit into
mainfrom
ci/repin-actions-main
Closed

ci: pin PredictiveEcology/actions to @main#30
eliotmcintire wants to merge 1 commit into
mainfrom
ci/repin-actions-main

Conversation

@eliotmcintire

Copy link
Copy Markdown
Contributor

Tags do not move, so a pinned tag freezes this repo on whatever the action looked like when that tag was cut.

v0.1 and v0.2 both predate the removal of the ubuntugis-unstable PPA, which installs libgdal37 while Posit's noble binaries are built against libgdal34 — an ABI mismatch that surfaces only at lazy-load, as libgdal.so.34: cannot open shared object file.

@main is the org standard: consumers pick up fixes without a pin bump, and PredictiveEcology/actions now has its own self-test CI gating what lands there (PredictiveEcology/actions#24).

Expect the first Ubuntu run to be slow, and possibly red once. Any cached R library built while the PPA was active contains a terra.so linked against libgdal.so.37, which no longer exists. I have already cleared this repo's Ubuntu caches where any existed; if a leg still fails with a missing libgdal.so.37, clear the Ubuntu caches and re-run — it is the one-time transition, not a regression.

🤖 Generated with Claude Code

https://claude.ai/code/session_012DVjmY3im9Xak7tXLSMGCa

Tags do not move, so a pinned tag freezes this repo on whatever the action
looked like when the tag was cut. v0.1/v0.2 predate the removal of the
ubuntugis-unstable PPA, which installs libgdal37 while Posit's noble binaries
are built against libgdal34 -- an ABI mismatch that surfaces only at lazy-load.

@main is the org standard: consumers follow fixes without a pin bump, and the
actions repo now has its own CI gating what lands there.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012DVjmY3im9Xak7tXLSMGCa
@eliotmcintire

Copy link
Copy Markdown
Contributor Author

Superseded by #31, which migrates this repo's R-CMD-check.yaml to the org reusable workflow.

This PR only moved the install-spatial-deps pin from a frozen tag to @main. The migration deletes that line outright — the reusable workflow installs the geospatial system dependencies itself via setup-r-deps, so there is no pin left to maintain here. Merging both would mean resolving a conflict for no gain.

Closing to keep the picture clear: any red on this PR reads as a migration failure and is not one.

🤖 Generated with Claude Code

https://claude.ai/code/session_012DVjmY3im9Xak7tXLSMGCa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant