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

updated yrs bindings to use version 0.17.2 #35

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

Horusiath
Copy link
Contributor

This PR updates yrs dependency to version 0.17.2.

@Horusiath Horusiath requested a review from nugmanoff December 11, 2023 09:00
@@ -16,7 +16,7 @@ unsafe impl Sync for YrsDoc {}
impl YrsDoc {
pub(crate) fn new() -> Self {
let mut options = Options::default();
options.offset_kind = OffsetKind::Utf32;
options.offset_kind = OffsetKind::Utf16;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We dropped support for Utf32, as it's wasn't used and it's not really useful. If Swift is using proper unicode, there are code points even longer than 4 bytes.

@heckj
Copy link
Collaborator

heckj commented Dec 11, 2023

The tuist stuff (what @nugmanoff originally set up to build Xcode projects for the examples) look to have bit-rotted as tuist evolved since we last updated and touched this, which is what's causing an issue with CI. I'll look into it more and see what I can sort out there (it's not something I'm deeply familiar with)...

I'm not having much luck directly, so I reached out to get help. This was previously working, so I'm not 100% clear on how it's evolved or what's changed to make it broken now: tuist/tuist#5671

@heckj heckj merged commit 12ada0f into y-crdt:main Dec 11, 2023
1 check passed
@nugmanoff
Copy link
Contributor

I guess I am late to the party. Just noticed the notifications. @heckj looks like you've succeeded with updating the Tuist syntax to the latest version – nice catch! Let me know if I can assist more

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.

3 participants