-
Notifications
You must be signed in to change notification settings - Fork 607
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
feat(encoding): added XML parser #2471
Conversation
This is a lot of work. Has there been a conversation with a core team member about this being a good contribution to |
I did not have any conversation about this. I was experimenting with XML and, since Deno has support for YAML, TOML, CSV, and other common formats, I thought it would be a nice addition. If you don't think it should be integrated in |
Is it possible to make this compatible with DOMParser API? |
Having XML parser as part of deno_dom would be a lot better in my opinion. YAML, TOML, CSV are far more simplistic of parsers than XML. |
I'm in favor of closing this one for now. I agree with @kitsonk . I think we should have xml parser as part of DOM API instead of invented API. |
To-do:
escape
utility