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

Add ability to set entity resolver when deserialize using borrowing reader #615

Merged
merged 5 commits into from
Jun 21, 2023

Conversation

Mingun
Copy link
Collaborator

@Mingun Mingun commented Jun 21, 2023

I noticed, that after #583 there is no way to create a deserializer that will borrow from string and provide ability to resolve custom entities. This PR fixes this gap.

@Mingun Mingun added enhancement serde Issues related to mapping from Rust types to XML labels Jun 21, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #615 (3342375) into master (0db486f) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #615      +/-   ##
==========================================
- Coverage   64.64%   64.62%   -0.02%     
==========================================
  Files          36       36              
  Lines       17003    16998       -5     
==========================================
- Hits        10991    10985       -6     
- Misses       6012     6013       +1     
Flag Coverage Δ
unittests 64.62% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/de/mod.rs 68.96% <100.00%> (-0.07%) ⬇️

... and 1 file with indirect coverage changes

Changelog.md Outdated
@@ -14,13 +14,15 @@

- [#609]: Added `Writer::write_serializable` to provide the capability to serialize
arbitrary types using serde when using the lower-level `Writer` API.
- [#615]: Added ability to set entity resolver when deserialize using borrowing reader.

### Bug Fixes

### Misc Changes


[#609]: https://github.com/tafia/quick-xml/issues/609
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be pull #609, not issue #609

My mistake, looks like I didn't fix it up completely.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, will fix. Just a moment...

Copy link
Collaborator

@dralley dralley left a comment

Choose a reason for hiding this comment

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

See changelog comment

@Mingun Mingun merged commit 60249ae into tafia:master Jun 21, 2023
6 checks passed
@Mingun Mingun deleted the borrowing-resolver branch June 21, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement serde Issues related to mapping from Rust types to XML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants