8000 fix: re-index childNodes on insertBefore by karfau · Pull Request #766 · xmldom/xmldom · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: re-index childNodes on insertBefore #766

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

Merged
merged 4 commits into from
Oct 26, 2024
Merged

Conversation

karfau
Copy link
Member
@karfau karfau commented Oct 23, 2024

to resolve a regression introduced in #760

fixes #763

to resolve a regression introduced in #760

fixes #763
@karfau karfau added this to the 0.9.5 milestone Oct 23, 2024
@karfau karfau self-assigned this Oct 23, 2024
Copy link
codecov bot commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.05%. Comparing base (5e56233) to head (2a4c6e5).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #766   +/-   ##
=======================================
  Coverage   95.05%   95.05%           
=======================================
  Files           8        8           
  Lines        2165     2165           
  Branches      569      569           
=======================================
  Hits         2058     2058           
  Misses        107      107           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mureinik
Copy link
mureinik commented Oct 25, 2024

FWIW, I can confirm that this PR solves not only the issue in #763, but also the original issue I faced in my work which #763 is a reduction thereof.

@karfau
Copy link
Member Author
karfau commented Oct 25, 2024
8000

That's good to know, I hope to finish this up and release it latest on Monday

@karfau karfau merged commit fa9df79 into master Oct 26, 2024
38 checks passed
@karfau karfau deleted the fix/insertBefore-appending branch October 26, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Milestone
0.9.5
Development

Successfully merging this pull request may close these issues.

insertBefore is appending the node in childNodes (0.9.4)
2 participants
0