Closed hikarunoryoma closed 1 year ago
LGTM, could you rebase?
LGTM, could you rebase?
Done! :)
Let me know if you want me to bump the version number. I was unsure how that is managed
There seems to be one reference left to fix:
> node ./node_modules/eslint/bin/eslint --ignore-path .gitignore '.'
/home/runner/work/web-push/web-push/test/test-vapid-helper.js
Error: 18:26 error 'urlBase64' is not defined no-undef
✖ 1 problem (1 error, 0 warnings)
Unfortunately it looks like the tests are failing
Thanks, I'll look to investigate this weekend and actually have JVM and run these test locally on my machine
@hikarunoryoma still planning on working on this?
when is this gonna get merged.
Thanks for the hard work
when is this gonna get merged.
Thanks for the hard work
There are some failing tests, they need to be fixed first before this can be merged.
i fixed the issue with decode @hikarunoryoma
Merged in #813, thank you!
Solves: https://github.com/web-push-libs/web-push/issues/785