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

feat(types): add most of type supports for the driver #48

Merged
merged 14 commits into from
Nov 2, 2024

Conversation

Daniel-Boll
Copy link
Owner

No description provided.

will be removed later

Signed-off-by: Daniel Boll <[email protected]>
This commit introduces support for Decimal and Duration types in the
Scylla driver. It also updates the query parameter and result parsing
to handle these new types. The changes are reflected in the TypeScript
definitions and JavaScript exports as well.

BREAKING CHANGE: The query parameter and result types have been updated
to accommodate the new Decimal and Duration types. This may require
changes in the calling code if it was relying on the previous types.

Signed-off-by: Daniel Boll <[email protected]>
Signed-off-by: Daniel Boll <[email protected]>
Floats, VarInts and Maps are not working currently. Also `Duration` and `Decimal` should have helper functions in their classes, but this will be implemented in another branch eventually.

Signed-off-by: Daniel Boll <[email protected]>
also added the generation of generics

Signed-off-by: Daniel Boll <[email protected]>
@Daniel-Boll Daniel-Boll self-assigned this Nov 2, 2024
@Daniel-Boll Daniel-Boll added the enhancement New feature or request label Nov 2, 2024
Copy link
Collaborator

@DanielHe4rt DanielHe4rt left a comment

Choose a reason for hiding this comment

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

congratz you deserve an açai geladinho

@Daniel-Boll Daniel-Boll merged commit 1513fad into main Nov 2, 2024
3 checks passed
@Daniel-Boll Daniel-Boll deleted the feat/result-types branch November 2, 2024 22:54
Daniel-Boll pushed a commit that referenced this pull request Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants