Repository navigation
Release 0.2.0 - #47
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
|
Claude finished @jnasbyupgrade's task in 15s —— View job Code review: PR #47 (Release 0.2.0)No blocking issues found. I checked the version bumps, the upgrade-script rename,
I did not build the extension or run the tests. I also did not compare the generated The post-merge steps in the PR description ( |
| @@ -0,0 +1,38 @@ | |||
| 0.2.0 | |||
| ----- | |||
| == Upgrading from 0.1.0 | |||
There was a problem hiding this comment.
Reads too much like directions on how to update, obscuring the backwards compatability break.
| `object__getsert()` raises `feature_not_supported` for any member of the | ||
| object_reference extension itself. | ||
|
|
||
| == Support cat_tools 0.3.0 |
There was a problem hiding this comment.
We now demand cat_tools 0.3.0, correct? Do we have code that checks that and provides a good error? Does META / control file specify >= 0.3.0? If not open a PR that does both.
There was a problem hiding this comment.
Oh, and this item needs to be reworded. We don't "support" 0.3.0, we demand it. That should be at the top of the list, along with backwards incompatabilities. Open a PR against ../ai/ that the history for a version must list any backwards incompatabilities or special steps needed to update at the top of what changed in a version.
|
Claude finished @jnasbyupgrade's task in 16s —— View job Code review: PR #47 (Release 0.2.0)One issue to fix.
As written, the release note would be false. This matches @jnasbyupgrade's review request: set the META minimum to 0.3.0 and add a version check that raises a clear error. The check needs to go in both Already addressed: the earlier Not done:
|
Bump version to 0.2.0, finalize the 0.1.0 -> 0.2.0 update script, add the generated 0.2.0 install script, and start HISTORY.asc. CLAUDE.md now points at the shared release process.
5268063 to
52dfa49
Compare
|
Claude finished @jnasbyupgrade's task in 13s —— View job Code review: PR #47 (Release 0.2.0)No blocking issues found. The earlier concerns are resolved.
Not done:
|
Version bump to 0.2.0 per the shared release process.
default_version,META.in.json(and regeneratedMETA.json) set to 0.2.0sql/object_reference--0.1.0--stable.sqlrenamed to--0.1.0--0.2.0.sql; generatedsql/object_reference--0.2.0.sqlcommittedHISTORY.ascwith the 0.2.0 changesCLAUDE.mdlinks the shared release processAfter merge:
make tag/make dist, upload to PGXN, then resetdefault_versiontostableand re-seedsql/object_reference--0.2.0--stable.sql.