Skip to content

Conversation

hyandell
Copy link

@hyandell hyandell commented Mar 9, 2021

Hi fast-xml-parser folk,

I noticed the license change back to MIT. I've updated the package.json and README to fit with that change; I hope that's okay.

@coveralls
Copy link

coveralls commented Mar 9, 2021

Coverage Status

Coverage remained the same at 87.983% when pulling 192841b on hyandell:master into da3d44c on NaturalIntelligence:master.

@guardrails
Copy link

guardrails bot commented Mar 9, 2021

⚠️ We detected security issues in this pull request:

Vulnerable Libraries (2)

More info on how to fix Vulnerable Libraries in Javascript.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

@amitguptagwl
Copy link
Member

@hyandell Yes the license is changed back to MIT original. However, I have kept the extra expectations as disclaimer under "License" on README.md. So let's keep it. Your change in package.json is fine.

@amitguptagwl
Copy link
Member

Changes are already published. So closing this PR. Thanks for your valuable effort.

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.

3 participants