Skip to content

Conversation

@xWyatt
Copy link
Contributor

@xWyatt xWyatt commented Jul 15, 2022

This PR updates documentation, documentation code samples, and the xmlToJson deprecation message to suggest the use of fast-xml-parser over xml2js.
Previous issues, confusion, and lack of maintenance of xml2js outlined in #344 lead to this PR.

@abmusse abmusse self-requested a review October 13, 2022 21:41
Copy link
Member

@abmusse abmusse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contributions!
Changes LGTM
Have you tested the updated code samples?

@abmusse
Copy link
Member

abmusse commented Oct 20, 2022

Can you sign-off the commits to appease the DCO bot

https://github.com/IBM/nodejs-itoolkit/pull/345/checks?check_run_id=7364528303

Thanks again!

@xWyatt xWyatt force-pushed the chore/docsReferenceAltXMLParser branch from a16102a to 99bcf95 Compare October 21, 2022 20:45
@xWyatt
Copy link
Contributor Author

xWyatt commented Oct 21, 2022

I tacked on sign-offs with my Github username to the commits as opposed to my profile name. That should appease the DCO bot - interesting that it won't accept the profile name in the first place though...

@abmusse abmusse self-requested a review October 26, 2022 20:41
Copy link
Member

@abmusse abmusse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abmusse abmusse merged commit 6fbb0a7 into IBM:master Oct 26, 2022
@abmusse abmusse mentioned this pull request Oct 26, 2022
3 tasks
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