Remove now-redundant local installcheck: install workaround - #33
Conversation
pgxntool 2.3.0 (Postgres-Extensions#31) adds this same installcheck: install edge directly in pgxntool/base.mk, fixing the check-stale-expected ordering issue this local copy worked around since Postgres-Extensions#9. Declaring it twice was harmless but is now dead weight. Co-Authored-By: Claude <noreply@anthropic.com>
|
Claude finished @jnasbyupgrade's task in 1m 13s —— View job Claude's Review
Verdict: LGTM. I verified the core claim in the PR description directly:
No issues found. This is a safe, well-justified cleanup. |
|
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 |
pgxntool 2.3.0 (#31) adds this same `installcheck: install` edge directly in `pgxntool/base.mk` (fixes the check-stale-expected ordering issue this local copy in the root `Makefile` worked around since #9). Declaring it twice was harmless to Make but is now dead weight, and the local comment's "until that's fixed upstream" is stale -- it's fixed in #31.
Flagged by the automated review on #31.
Co-Authored-By: Claude noreply@anthropic.com