-
Notifications
You must be signed in to change notification settings - Fork 173
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
Exports C14nCanonicalization, ExclusiveCanonicalization #336
Conversation
Codecov Report
@@ Coverage Diff @@
## master #336 +/- ##
=======================================
Coverage 82.88% 82.88%
=======================================
Files 7 7
Lines 777 777
=======================================
Hits 644 644
Misses 133 133 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@angeljqv can you lint this and then we can merge it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@angeljqv - please run npm run lint:fix
on your code to make your code acceptable, so that our automated tests pass
hi @cjbarth, this PRs has no effect on v4 |
@parallels999 , are you identifying a problem or concern? I don't understand the nature of your comment. |
Lines 23 to 25 in 6e6a8b0
Just concern, i am reading v4 changelog, and this is not available in v4, but it works in v3 |
That is an oversight (or bad merge) then, sorry about that. Feel free to put up a PR against |
Same changes as #335 on to
master
.