Skip to content

chore(deps): pin pypgstac to 0.9.11 to match pgstac v0.9.11 image - #87

Merged
pantierra merged 1 commit into
mainfrom
fix/ci-pgstac-version
Sep 7, 2026
Merged

chore(deps): pin pypgstac to 0.9.11 to match pgstac v0.9.11 image#87
pantierra merged 1 commit into
mainfrom
fix/ci-pgstac-version

Conversation

@pantierra

Copy link
Copy Markdown
Contributor

No description provided.

@claude

claude Bot commented Sep 7, 2026

Copy link
Copy Markdown

Claude finished @pantierra's task in 54s —— View job


Reviewing PR #87

  • Gather context (diff, CLAUDE.md)
  • Review changes for correctness/security issues
  • Apply ponytail-review pass
  • Post final review

✅ No blocking issues — safe to merge.

The diff pins pypgstac back to 0.9.11 (pyproject.toml:22, uv.lock) to match the ghcr.io/stac-utils/pgstac:v0.9.11 image pinned in docker-compose.yml:216, and switches dockerfiles/Dockerfile.browser / docker-compose.yml from the pantierra/stac-browser fork back to upstream radiantearth/stac-browser now that PR #930 merged there. Both changes are internally consistent — client and server pgstac schema versions now match, and the browser image references a real upstream commit instead of a fork ref.

Non-blocking notes:

  • The PR title says "upgraded pypgstac to version 0.9.12", but the actual change (per the commit message) pins it back to 0.9.11 to match the pgstac image version — the title/description doesn't match the intent. Worth correcting before merge so the commit history reads clearly.
  • .github/workflows/ingest.sh:33,36 separately pins pypgstac==0.9.9 as its fallback install path, which is now inconsistent with the 0.9.11 pinned everywhere else. Pre-existing and out of scope for this PR, but worth a follow-up so CI/local ingest and the schema stay aligned.

Simplify (ponytail): nothing to flag — this is a pure version-pin/dependency diff with no new code or abstractions.


💰 Estimated review cost: $0.26 · 0m53s · 17 turns

@pantierra
pantierra force-pushed the fix/ci-pgstac-version branch from 86e8ed4 to 92825de Compare September 7, 2026 15:22
The database service in docker-compose.yml/ci.yml pins the pgstac
image to v0.9.11. A prior dependency bump moved pypgstac to 0.9.12,
whose client queries partition_catalog_meta(), a function only
present in the v0.9.12 schema, breaking CI's pypgstac load step.
Also switches the stac-browser build from the pantierra fork back
to the upstream radiantearth/stac-browser repo now that PR #930 has
merged there, since the old fork commit is no longer reachable.
@pantierra
pantierra force-pushed the fix/ci-pgstac-version branch from 92825de to 7c762da Compare September 7, 2026 15:28
@pantierra pantierra changed the title chore(deps): upgraded pypgstac to version 0.9.12. chore(deps): pin pypgstac to 0.9.11 to match pgstac v0.9.11 image Sep 7, 2026
@pantierra
pantierra merged commit b3b341d into main Sep 7, 2026
3 checks passed
@pantierra
pantierra deleted the fix/ci-pgstac-version branch September 7, 2026 15:30
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