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

Implementation and tests for ordering of element attributes #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PAX523
Copy link

@PAX523 PAX523 commented May 22, 2019

This behavior is optional.

@codecov-io
Copy link

codecov-io commented May 22, 2019

Codecov Report

Merging #69 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #69      +/-   ##
============================================
+ Coverage     39.93%   39.99%   +0.05%     
- Complexity     1493     1499       +6     
============================================
  Files           145      145              
  Lines          9490     9497       +7     
  Branches       1192     1193       +1     
============================================
+ Hits           3790     3798       +8     
+ Misses         5320     5319       -1     
  Partials        380      380
Impacted Files Coverage Δ Complexity Δ
src/main/java/org/dom4j/io/OutputFormat.java 63.63% <100%> (+1.56%) 32 <4> (+4) ⬆️
src/main/java/org/dom4j/io/XMLWriter.java 60.92% <100%> (+0.1%) 155 <8> (+1) ⬆️
...rc/main/java/org/dom4j/tree/ContentListFacade.java 29.78% <0%> (+2.12%) 8% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8073066...d70159c. Read the comment docs.

@FilipJirsak FilipJirsak self-assigned this Jan 22, 2023
@FilipJirsak FilipJirsak added this to the 2.2.0 milestone Jan 22, 2023
@long76
Copy link

long76 commented Apr 24, 2024

@FilipJirsak please merge this, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants