Skip to content

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
@codecov
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

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

Development

Successfully merging this pull request may close these issues.

insertBefore is appending the node in childNodes (0.9.4)

2 participants