Repository navigation
chore: remove schema files until their design is settled - #6
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
No configured credential can serve this run. Your Claude Pro/Max subscription was rejected. Sign in again by running The providers' responses
|
Contributor
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
The branch had no earlier Pullfrog review, so this run covers the new merge commit and the full revert.
- Merged
maincleanly:git show --cc 75ea55dshows no conflict resolutions. #8's MySQLVECTORchanges come in unchanged. - Rebuilt
packages/mysql/engine.wasm: the new artifact contains #8'sto_vector(rewrite. It no longer has theschemakey's"a path, or a list of paths"deserializer message, whichmain's artifact still has. The Postgres and SQLite artifacts lack that message as well. - Removed the
schemakey end to end: this coversconfig.rs,analyze.ts,cli.ts,mapper.ts,project.tsand the docs. Nothing insrc/ortests/still refers toschema-files.ts. The rewritteninclude/excludefilter ingeneratebehaves the same as before for projects that set neither key. - Dropped
@electric-sql/pglitefrom the root package:tests/representation.test.tsstill loads PGlite, but it resolves it throughlab/drivers/node_modules, so the removal doesn't break it.
claude-opus-5-5 ｜ 𝕏
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Sugges
2E41
tions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes the
schemakey of.reylo.yaml, which typed queries from SQL files (a schema dump or a directory of migrations) with no database, while its design is reconsidered: running the project's own migration command against a throwaway database may replace reading migration files directly.schemakey,src/schema-files.ts, its tests and fixture, the PGlite dev dependency and optional peer, and the docs.engine.wasmartifacts from the reverted source, since a9ab0dd rebuilt them after the key was added.