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
bump xmldom to 0.6.0 to avoid bug where it keeps "<" in attributes #1974
Conversation
adding a test where a schema with |
@calebmshafer should this be backported? |
@calebmshafer I'm getting flaky integration and standalone backend tests, could you take a look? |
@skirby1996 is working on the extract-api issue. There's some combination of node version and something else causing a bunch of people issues. The The flaky integration tests, I have not looked into those yet so not quite sure. |
@calebmshafer when we get past that stuff, can we also backport this fix to 2.19? |
@Mergifyio update |
Command
|
@Mergifyio backport release/2.19.x |
…1974) * bump xmldom to 0.6.0 to avoid bug where it keeps "<" in attributes see xmldom/xmldom#198 * add test which fails on version of 0.5.0 because of bad serialization * add valid entity serialization test * move serialization test down to serializer package instead of up at transformer level * better test name not referencing round trip Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com> (cherry picked from commit 97a229f) # Conflicts: # common/config/rush/pnpm-lock.yaml
Command
|
…ackport #1974) (#1994) * bump xmldom to 0.6.0 to avoid bug where it keeps "<" in attributes (#1974) * bump xmldom to 0.6.0 to avoid bug where it keeps "<" in attributes see xmldom/xmldom#198 * add test which fails on version of 0.5.0 because of bad serialization * add valid entity serialization test * move serialization test down to serializer package instead of up at transformer level * better test name not referencing round trip Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com> (cherry picked from commit 97a229f) # Conflicts: # common/config/rush/pnpm-lock.yaml * pushed merged pnpm-lock Co-authored-by: Michael Belousov <mike.belousov@bentley.com> Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>
…1974) * bump xmldom to 0.6.0 to avoid bug where it keeps "<" in attributes see xmldom/xmldom#198 * add test which fails on version of 0.5.0 because of bad serialization * add valid entity serialization test * move serialization test down to serializer package instead of up at transformer level * better test name not referencing round trip Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>
…1974) * bump xmldom to 0.6.0 to avoid bug where it keeps "<" in attributes see xmldom/xmldom#198 * add test which fails on version of 0.5.0 because of bad serialization * add valid entity serialization test * move serialization test down to serializer package instead of up at transformer level * better test name not referencing round trip Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>
see xmldom/xmldom#199 and xmldom/xmldom#198
Some known schemas use
<
and>
in display labels, so this was breaking the imodel transformer which usedxmldom@0.5.0
to serialize schemas during export to be imported into the target iModel.