Revert CAT_TOOLS_GIT_REF override now that cat_tools 0.3.0 is on PGXN - #47
Revert CAT_TOOLS_GIT_REF override now that cat_tools 0.3.0 is on PGXN#47jnasbyupgrade wants to merge 1 commit into
Conversation
cat_tools 0.3.0 published to PGXN 2026-08-06, so a plain `pgxn install` now satisfies META.in.json's declared floor -- the git-source override this repo's (now-deleted) RELEASE.md warned never to release with is no longer needed. Unblocks cutting the 1.0.0 release per ../ai/RELEASE.md. pg-tle-test still needs a concrete git ref for its own from-scratch cat_tools clone (that job never installs via PGXN), so it gets its own job-level CAT_TOOLS_GIT_REF, decoupled from the removed release gate.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Claude finished @jnasbyupgrade's task in 1m 6s —— View job Code ReviewTasks:
SummaryThis is exactly the release-blocking cleanup the PR description says it is, and the diff backs it up:
No issues found. Nothing to flag, no suggested fixes. |
Summary
pgxn installnow satisfies
META.in.json's declaredcat_tools: 0.3.0floor. Theworkflow-level
CAT_TOOLS_GIT_REF: 0.3.0override (added to work aroundPGXN's previously-stale 2017
cat_toolsrelease) is no longer needed.RELEASE.mdand the shared../ai/RELEASE.mdwarn about: never cut arelease while a dependency is pinned to a git ref instead of PGXN. Clearing
it is a prerequisite for cutting the pending 1.0.0 release.
pg-tle-teststill needs a concrete git ref for its own from-scratchcat_toolsclone (that job registers cat_tools viapg_tle, never throughPGXN/filesystem install), so it now carries its own job-level
CAT_TOOLS_GIT_REF: 0.3.0, decoupled from the removed top-level override.RELEASE.md: its only remaining content was documentingthis one instance of the override, which this PR resolves. The generic
guardrail is already covered by the shared
Postgres-Extensions/ai/RELEASE.md(linked from the file being removed) — per that doc's own guidance, a repo
with no other repo-specific deviation doesn't need a local copy.
Verified locally
pgxn install 'cat_tools>=0.2.1' --sudofetches and installs
cat_tools0.3.0 from PGXN (confirmed by running it).make verify-resultspasses on PG17 (fresh install + build tests)..github/workflows/ci.ymlparses as valid YAML after the edit.Test plan
confirms the plain-PGXN path works in the actual CI container, not just
locally.