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

Delete xmltree/indexmap dependencies #2111

Merged
merged 2 commits into from
Nov 1, 2023
Merged

Conversation

jcamiel
Copy link
Collaborator

@jcamiel jcamiel commented Nov 1, 2023

Due to xmltree re-exposing an older version of indexmap, we couldnt' upgrade to the latest version of indemap.
xmltree provides an in-memory tree representation of an XML document that we use for JUnit report. As xmltree is a thin layer above xml-rs, we re-implement a tree in-memory XML document using xml-rs directly and remove xmltree/indexmap dependencies.

Due to xmltree re-exposing an older version of indexmap, we couldnt' upgrade to the latest version of indemap.
xmltree is a tree in-memory representation of an XML document that we use for JUnit export. As xmltree is a thin layer above xml-rs, we re-implement a thin tree in-memory XML document using xml-rs directly and remove xmltree/indexmap dependency.
@jcamiel jcamiel changed the title Delete xmltree rs dependency Delete xmltree/indexmap dependencies Nov 1, 2023
@jcamiel jcamiel self-assigned this Nov 1, 2023
@jcamiel
Copy link
Collaborator Author

jcamiel commented Nov 1, 2023

/accept

@hurl-bot
Copy link
Collaborator

hurl-bot commented Nov 1, 2023

🕗 /accept is running, please wait for completion.

@hurl-bot
Copy link
Collaborator

hurl-bot commented Nov 1, 2023

✅ Pull request merged and closed by jcamiel with fast forward merge..

# List of commits merged from Orange-OpenSource/hurl/delete-xmltree-rs-dependency branch into Orange-OpenSource/hurl/master branch:

  • c72f04f Remove xmltree/indexmap dependency.
  • 842a167 Add DOM XML parser based on xml-rs.

@hurl-bot hurl-bot merged commit c72f04f into master Nov 1, 2023
19 checks passed
@hurl-bot hurl-bot deleted the delete-xmltree-rs-dependency branch November 1, 2023 19:56
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.

2 participants