Skip to content

Conversation

@rkistner
Copy link
Contributor

We have been recommending this syntax for a while - the generator should use this by default.

We can still give the option of the old syntax for apps not wanting to switch.

The generator here includes the entire "schema" file, including imports and exports.

@changeset-bot
Copy link

changeset-bot bot commented Aug 14, 2024

🦋 Changeset detected

Latest commit: 756ad8c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@powersync/service-sync-rules Patch
@powersync/service-core Patch
@powersync/service-image Patch
test-client Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@rkistner rkistner requested a review from DominicGBauer August 14, 2024 15:02
DominicGBauer
DominicGBauer previously approved these changes Aug 14, 2024
DominicGBauer
DominicGBauer previously approved these changes Aug 15, 2024
@rkistner rkistner requested a review from DominicGBauer August 15, 2024 11:05
Copy link
Contributor

@DominicGBauer DominicGBauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a test failed

@rkistner rkistner merged commit 5758e4c into main Aug 15, 2024
@rkistner rkistner deleted the tablev2-schema-generator branch August 15, 2024 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants