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.

Sign up for GitHub

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

Jump to bottom

remove klaxon usage outside json.kt #506

Merged

koperagen merged 1 commit into master from klaxon-fix

Nov 17, 2023

Collaborator

koperagen commented Nov 17, 2023

Excluding klaxon transivite dependency which is fine if you don't use JSON reading can lead to an unexpected class not found exception now


          remove klaxon usage outside json.kt

8cb4030

koperagen requested a review from Jolanrensen

November 17, 2023 13:56

Jolanrensen approved these changes

View reviewed changes

Collaborator

Jolanrensen left a comment

Lol, indeed, we don't need klaxon for this single function

koperagen merged commit 9d65788 into master

2 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment