-
Notifications
You must be signed in to change notification settings - Fork 15
Adding pure Java support of V2 Local token types #12
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
Conversation
Codecov Report
@@ Coverage Diff @@
## main #12 +/- ##
============================================
+ Coverage 83.89% 84.39% +0.50%
- Complexity 265 285 +20
============================================
Files 56 58 +2
Lines 770 846 +76
Branches 55 55
============================================
+ Hits 646 714 +68
- Misses 94 101 +7
- Partials 30 31 +1
Continue to review full report at Codecov.
|
Is this till underway? |
@bwoods8 yup! I’ll take another look at it on Monday. If you want, try out the brand and give us feed back! |
Rebased (and squashed) |
Looks good and worked like a charm! Thanks! 😄 |
Is this still in the works? 😄 |
Yes! thanks for the bump! I'll rebase the PR and get it moving! |
c721045
to
5ab7a3f
Compare
* Support for libsodium may be removed in the future * Updated README to mention v2.local support with the Bouncy Castle module * Refactored ITs, instead of running the ITs once, they now run as part of of the individual module builds - This ensure the ITs are run for each module (instead of just the first one on the classpath) - The shared tests are now fall under `main/src/groovy` so they can be included and run from the other modules.
5ab7a3f
to
6c14669
Compare
@bwoods8 in and released |
That's great news for the weekend! 😄 |
Refactored ITs, instead of running the ITs once, they now run as part of of the individual module builds
main/src/groovy
so they can be included and run from the other modules.