Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(arktype): export on model to arktype #49

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

BalogunofAfrica
Copy link
Contributor

The compile method on the scope object has been renamed to export in newer versions (2.x.x) of arktype.

@sinclairzx81
Copy link
Owner

@BalogunofAfrica Hi, thanks for the PR (and sorry for the delay on this)

Everything looks good here. However I did note there appears to be an issue https://sinclairzx81.github.io/typebox-workbench/ when inferring the type from a ark scope. I've tried updating monaco to the latest version (getting a newer version of the compiler running) but still having problems, so not too sure there.

Will merge this through anyway as the updates here are correct and inline with arktype 2.x (meaning the generation is correct). Will chase up monaco configurations on the Workbench project later :) Good work!

CC @ssalbdivad

@sinclairzx81 sinclairzx81 merged commit 7179b69 into sinclairzx81:main Aug 9, 2024
3 checks passed
@sinclairzx81 sinclairzx81 mentioned this pull request Aug 9, 2024
@sinclairzx81
Copy link
Owner

@BalogunofAfrica Hiya

Updates published on 0.10.4. Can previous them https://sinclairzx81.github.io/typebox-workbench/

Thanks again!
S

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.

2 participants