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

Docs: Reference alternate XML parser #345

Merged
merged 4 commits into from
Oct 26, 2022

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!

Signed-off-by: Wyatt Teeter <teeterwyatt@gmail.com>
Signed-off-by: xWyatt <wteeter@ppi-global.com>
…uide

Signed-off-by: Wyatt Teeter <teeterwyatt@gmail.com>
Signed-off-by: xWyatt <wteeter@ppi-global.com>
Signed-off-by: Wyatt Teeter <teeterwyatt@gmail.com>
Signed-off-by: xWyatt <wteeter@ppi-global.com>
Signed-off-by: Wyatt Teeter <teeterwyatt@gmail.com>
Signed-off-by: xWyatt <wteeter@ppi-global.com>
@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.

None yet

2 participants