-
Notifications
You must be signed in to change notification settings - Fork 18
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
Node16 support #236
Node16 support #236
Conversation
9b8dfae
to
3b3dbfb
Compare
Codecov Report
@@ Coverage Diff @@
## master #236 +/- ##
==========================================
- Coverage 82.65% 82.47% -0.18%
==========================================
Files 80 79 -1
Lines 1753 1735 -18
Branches 418 414 -4
==========================================
- Hits 1449 1431 -18
Misses 303 303
Partials 1 1
Continue to review full report at Codecov.
|
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.
Looks good to me. 👍
f675d06
to
a7242f2
Compare
|
||
- name: Setup npm | ||
run: npm install -g npm@6 |
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.
Due to dependency conflicts during installations with Npm 8.6.0 as mentioned in this Issue we are pinning it to 8.5.x for now
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.
LGTM 👍
No description provided.