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

Bump to latest rust for streaming improvements #270

Merged
merged 3 commits into from
Mar 1, 2024

Conversation

nplasterer
Copy link
Contributor

This bumps to the latest rust versions to get the latest streaming improvements.

@nplasterer nplasterer self-assigned this Feb 29, 2024
@nplasterer nplasterer requested a review from a team as a code owner February 29, 2024 19:10
@@ -44,5 +44,5 @@ Pod::Spec.new do |spec|
spec.dependency "web3.swift"
spec.dependency "GzipSwift"
spec.dependency "Connect-Swift", "= 0.3.0"
spec.dependency 'LibXMTP', '= 0.4.2-beta3'
spec.dependency 'LibXMTP', '= 0.4.2-beta5'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI @zombieobject we really shouldn't have the cocoapod and the SPM pointing to different versions of libxmtp.

Copy link
Contributor

Choose a reason for hiding this comment

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

Noted. I will be sure to bump both at the same time going forward.

@nplasterer
Copy link
Contributor Author

Looks like these test failures might be legit. Streaming conversations seems to not be working as expected.

@zombieobject zombieobject added on-hold Do not merge. This PR is on hold. and removed on-hold Do not merge. This PR is on hold. labels Mar 1, 2024
@zombieobject zombieobject force-pushed the np/bump-latest-rust branch from 229c1ae to c9c14a2 Compare March 1, 2024 20:50
Copy link
Contributor

@zombieobject zombieobject left a comment

Choose a reason for hiding this comment

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

The test runs actually passed after another run. I will be updating the workflow to retry in a future PR.

@zombieobject zombieobject merged commit 990f4af into main Mar 1, 2024
2 checks passed
@zombieobject zombieobject deleted the np/bump-latest-rust branch March 1, 2024 21:01
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