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

chore: update sync_wrapper to 1.0 #2281

Merged
merged 1 commit into from May 20, 2024

Conversation

lucab
Copy link
Contributor

@lucab lucab commented May 10, 2024

This updates sync_wrapper to its latest version (1.0.1).

Ref: Actyx/sync_wrapper@v0.1.2...v1.0.1

@lucab
Copy link
Contributor Author

lucab commented May 10, 2024

For reference, there is an ongoing discussion at #2229 about whether to drop this dependency or wire it properly in the codebase.

Regardless of that, I currently have a consumer with multiple version of sync_wrapper in its lockfile. Thus in the short term I'd like to get rid of the stale copy coming as reqwest transitive dependency.

This updates `sync_wrapper` to its latest version (`1.0.1`).

Ref: Actyx/sync_wrapper@v0.1.2...v1.0.1
@lucab lucab force-pushed the ups/cargo-sync_wrapper-1.0 branch from 0249c0b to bd7e887 Compare May 20, 2024 10:21
@lucab
Copy link
Contributor Author

lucab commented May 20, 2024

@seanmonstar if you have some time to review it, I'd be happy to have this one merged.

@seanmonstar seanmonstar merged commit 1f951d0 into seanmonstar:master May 20, 2024
34 checks passed
@lucab lucab deleted the ups/cargo-sync_wrapper-1.0 branch May 20, 2024 17:09
@lucab
Copy link
Contributor Author

lucab commented May 20, 2024

@seanmonstar thanks a lot! ❤️

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.

None yet

2 participants