feat(database): add write operation capabilities - #194
QuintinWillison wants to merge 2 commits into
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 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 |
QuintinWillison
left a comment
There was a problem hiding this comment.
Bit awkward that I can only comment rather than request changes as the "author" of the PR, but this seemed to be the best mechanism for me to sponsor this change in...
|
Thanks @QuintinWillison for sponsoring this PR and taking the time to review it. Both suggestions make sense. I'll correct the bulk-payload prerequisite and move the capability below insert, then run the package checks before pushing the update. |
|
Both review suggestions are addressed and pushed in c054913. On Linux with Node 22, all 214 tests passed, along with typechecking, matrix validation, report generation, and site generation. I also checked the generated links for the three new capabilities and their related specs. The Windows suite hit path-specific failures in unchanged Python/Swift parser tests, so the full suite was run on Linux rather than modifying those tests. The working tree is clean. |
I'm sponsoring this pull request because @dhruvxvaishnav is unable to create pull requests in this repository.
Closes #113
🤖 Supabase process notes:
At merge: keep the auto-added
Co-authored-by: dhruvxvaishnav <dhruvvaishnav687@gmail.com>trailer in the squash message (that is what carries their contribution credit under squash), re-verify the head SHA is the one reviewed, and approve workflow runs per push after reading the diff.