Skip to content

Upgrade ruma #76

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jul 26, 2020
Merged

Upgrade ruma #76

merged 1 commit into from
Jul 26, 2020

Conversation

jplatte
Copy link
Collaborator

@jplatte jplatte commented Jul 26, 2020

By the way, you've got a weird mix of one use statement per crate and one use statement per module. And also a mix of matrix_sdk_common imports and crate imports in matrix_sdk_base. You may want to fix that.

@jplatte
Copy link
Collaborator Author

jplatte commented Jul 26, 2020

Most of these changes could have been done earlier, but due to Rust not warning about deprecated re-exports (rust-lang/rust#47236), I forgot them. Now I removed those deprecated re-exports since the next releases will be breaking change releases anyway (would have been nice to keep the aliases around for a while, but not without deprecation warnings).

@poljar
Copy link
Contributor

poljar commented Jul 26, 2020

By the way, you've got a weird mix of one use statement per crate and one use statement per module. And also a mix of matrix_sdk_common imports and crate imports in matrix_sdk_base. You may want to fix that.

Yeah, one of these days I'll get to fixing that, most of this is left over since we first had only once crate and no matirx_sdk_common crate.

Copy link
Contributor

@poljar poljar left a comment

Choose a reason for hiding this comment

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

Thanks.

@poljar poljar merged commit 6a96368 into matrix-org:master Jul 26, 2020
@jplatte jplatte deleted the upgrade-ruma-3 branch July 26, 2020 19:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants