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

upgrade quick-xml #23

Closed
wants to merge 1 commit into from
Closed

Conversation

jobafr
Copy link

@jobafr jobafr commented Nov 13, 2024

Pulling in some bugfixes since the previous version, adjusting for a small API change on serialization.

The current crates-io version of dxr still uses quick-xml 0.30.x, which has a bug in its expand_empty_elements behavior that we're hitting in ros-core-rs (which depends on dxr). It would be really cool if you could do a crates.io release which has a newer quick-xml, so we could depend on that one :)

@decathorpe
Copy link
Member

Thank you - I had been planning to make a new release soon, though there are still some pending changes I need to make (notably in multicall support code) before that can happen. I'll try to allocate some time for that in the next few days.

@decathorpe
Copy link
Member

I pushed some fixes for new clippy warnings and formatting issues and then applied the quick-xml 0.36 -> 0.37 update directly, I hope that's alright with you for a two-line change.

I still need to fix up multicall support before I can publish a new release, but I still hope that will be soon. 😞

@decathorpe decathorpe closed this Nov 30, 2024
@decathorpe
Copy link
Member

I have decided to postpone multicall client API changes in favor of getting a new release out sooner - 0.7.0 is now published. Enjoy!

@jobafr
Copy link
Author

jobafr commented Dec 2, 2024

Thanks :)

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.

2 participants