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

Update Rune version to 0.13 #96

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Update Rune version to 0.13 #96

merged 1 commit into from
Aug 6, 2024

Conversation

pkolaczk
Copy link
Owner

@pkolaczk pkolaczk commented Aug 6, 2024

No description provided.

@pkolaczk pkolaczk merged commit 30491eb into main Aug 6, 2024
4 checks passed
@pkolaczk pkolaczk deleted the upgrade-rune branch August 6, 2024 11:55
vponomaryov added a commit to scylladb/latte that referenced this pull request Oct 24, 2024
In the PR [1] with the switch from 'rune-0.12' to 'rune-0.13'
the 'blob' function changed it's return type from 'rune::runtime::Bytes'
to the 'Vec<u8>'.
Such a change required also an update to the input values parser
to expect one more input type match to the 'ColumnType::Blob'.

So, add the missing parsing element to make the 'blob' function work
again as it was before.
It will allow to avoid manual wrapping into the
'rune::runtime::Bytes' type in rune scripts.

[1] pkolaczk#96
vponomaryov added a commit to scylladb/latte that referenced this pull request Nov 15, 2024
In the PR [1] with the switch from 'rune-0.12' to 'rune-0.13'
the 'blob' function changed it's return type from 'rune::runtime::Bytes'
to the 'Vec<u8>'.
Such a change required also an update to the input values parser
to expect one more input type match to the 'ColumnType::Blob'.

So, add the missing parsing element to make the 'blob' function work
again as it was before.
It will allow to avoid manual wrapping into the
'rune::runtime::Bytes' type in rune scripts.

[1] pkolaczk#96
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.

1 participant