-
Notifications
You must be signed in to change notification settings - Fork 14
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
[v2] Update workflows and deps for technical currency #170
Conversation
Signed-off-by: Timothy Johnson <[email protected]>
Signed-off-by: Timothy Johnson <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## zowe-v2-lts #170 +/- ##
============================================
Coverage 75.22% 75.22%
============================================
Files 25 25
Lines 331 331
Branches 41 42 +1
============================================
Hits 249 249
Misses 82 82 ☔ View full report in Codecov by Sentry. |
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.
Tested building and installing the plugin with Node 22 and LGTM, thanks for the updates!
Signed-off-by: Timothy Johnson <[email protected]>
Quality Gate passedIssues Measures |
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! 😋
This PR didn't release because the v2 branch was nor in the release config file Once we merge the PR below, we should be fine: |
Release succeeded for the The following packages have been published:
Powered by Octorelease 🚀 |
What It Does
Also updates ibm_db dependency to support Node.js 22 on Windows (w/o enhancement for M1 Macs)
How to Test
Review Checklist
I certify that I have:
Additional Comments