Closed nicolo-ribaudo closed 2 years ago
Great job at this @nicolo-ribaudo 🎉
Also, thanks for testing it @adbayb, we appreciate it!
Thanks for this update! Any estimate on when this will be completed?
@Elgeneinar I'm scheduling the release to be done between today and tomorrow, but I'll try to let everyone knows in all following corresponded PRs
Description
I wrote the description of this fix and the reason behind it at https://github.com/doczjs/docz/issues/1691#issuecomment-1025928444.
I fixed it in different commits:
test(docz-utils): add test for current removeTags behavior
shows the old behavior (one test was correct, the other one was already broken)chore(docz-utils): update @babel/* dependencies
updates@babel/traverse
and the other babel packages, to force the version containing the fix for the bug docz was relying ontest(docz-utils): update removeTags (broken) snapshots
updates the tests to show the new broken behaviorfix(docz-utils): fix closing tag detection in removeTags
fixes both the broken testsFixes #1691