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

Make Transit an optional dependency #130

Open
Deraen opened this issue Sep 6, 2022 · 0 comments
Open

Make Transit an optional dependency #130

Deraen opened this issue Sep 6, 2022 · 0 comments

Comments

@Deraen
Copy link
Member

Deraen commented Sep 6, 2022

Transit format is now included in the default options and required on the core ns. AOT compiling Muuntaja core will also compile the Transit library.

User could exclude the dependency and add a local namespace to override muuntaja.format.transit with no-op format declarations. Even this is somewhat inconvenient as those formats need to return Encoder/Decoder instances for the default instance.

Transit isn't a big dependency, but in some environments, like Azure Function/AWS Lambda, everything extra on the classpath makes the startup slower.

We could separate the core and some easy ns which includes options/instance with Transit and others.

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

No branches or pull requests

1 participant