-
Notifications
You must be signed in to change notification settings - Fork 21
Remove quick-xml
warning by updating dependencies
#26
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
base: master
Are you sure you want to change the base?
Remove quick-xml
warning by updating dependencies
#26
Conversation
related to this discussion.. #21 |
@AlecTroemel is there anything we can do to help move things forward regarding quick-xml ? I noticed there are 2 PRs open, this one looks like the most painless, do we need to do some testing ? |
@mrguiman , I think I am still a co-maintainer on this one and can try merging. My understanding is that this crate was superseded by newer, better ways of doing the conversion and should probably be archived. Do you think we should keep it going? |
@rimutaka that would be up to the maintainers I guess. If there are indeed alternatives and no plan on updating this crate then it wouldn't be to much pain for me to switch, but then yes the repo should probably be archived and the readme updated to point towards said alternatives ? |
@John2143, @mrguiman, and @rimutaka I've had basically zero time to work on this project, but I should have still been more communicative. I apologize. The internal debate I'm having is, like @rimutaka mentioned, it seems like the rust ecosystem has move on from @rimutaka if there's a suitable alternative I think archiving would be reasonable. |
@AlecTroemel Using plain I personally don't care much if it is using |
Hello!
This PR updates the dependencies in
Cargo.toml
, specificallyminidom v0.12 -> v0.15
. I encountered the following warning when compiling on version0.6.0
:I tried updating the dependencies and it seems that newer version of minidom have dropped quick-xml entirely.