Skip to content

Update diff#2666

Merged
t1m0thyj merged 5 commits intomasterfrom
update-diff
Jan 27, 2026
Merged

Update diff#2666
t1m0thyj merged 5 commits intomasterfrom
update-diff

Conversation

@awharn
Copy link
Copy Markdown
Member

@awharn awharn commented Jan 15, 2026 •
edited
Loading

Signed-off-by: Andrew W. Harn andrewwilliamharn@gmail.com

What It Does

How to Test

Review Checklist
I certify that I have:

  • updated the changelog
  • manually tested my changes
  • added/updated automated unit/integration tests
  • created/ran system tests (provide build number if applicable) [1848]
  • followed the contribution guidelines

Additional Comments

@awharn
Update diff
d28f3d0 
Signed-off-by: Andrew W. Harn <andrewwilliamharn@gmail.com>
@github-project-automation github-project-automation Bot moved this to New Issues in Zowe CLI Squad Jan 15, 2026
@zowe-robot zowe-robot moved this from New Issues to Review/QA in Zowe CLI Squad Jan 15, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 15, 2026 •
edited
Loading

📅 Suggested merge-by date: 2/10/2026

@awharn
Update changelog with PR link
85b3540 
Signed-off-by: Andrew W. Harn <andrewwilliamharn@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 15, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.84%. Comparing base (2a4169a) to head (3069f04).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files 
@@           Coverage Diff           @@
##           master    #2666   +/-   ##
=======================================
  Coverage   91.84%   91.84%           
=======================================
  Files         645      645           
  Lines       19281    19281           
  Branches     4290     4290           
=======================================
  Hits        17708    17708           
  Misses       1571     1571           
  Partials        2        2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@awharn
Copy link
Copy Markdown
Member Author

awharn commented Jan 15, 2026

This PR is on hold til diff2html is updated.

@awharn awharn marked this pull request as draft January 15, 2026 21:30
@zowe-robot zowe-robot moved this from Review/QA to In Progress in Zowe CLI Squad Jan 15, 2026
zFernand0
zFernand0 approved these changes Jan 16, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@zFernand0 zFernand0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 😋

@github-project-automation github-project-automation Bot moved this from In Progress to Review/QA in Zowe CLI Squad Jan 16, 2026
@traeok
Copy link
Copy Markdown
Member

traeok commented Jan 16, 2026

This PR is on hold til diff2html is updated.

PR w/ diff updates is not yet merged but linking for tracking purposes:
rtfpessoa/diff2html#576

@JTonda JTonda added the Blocked label Jan 26, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 27, 2026

Reminder: This pull request has a merge-by date coming up within the next 24 hours. Please review this PR as soon as possible.

@t1m0thyj @traeok @jace-roell @ATorrise

awharn added 2 commits January 27, 2026 09:53
@awharn
Merge remote-tracking branch 'origin/master' into update-diff
1445d5b 
Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
@awharn
Update packages
674c91a 
Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
@awharn awharn marked this pull request as ready for review January 27, 2026 16:03
@awharn
Copy link
Copy Markdown
Member Author

awharn commented Jan 27, 2026

Marking as ready for review since we want to get the release out ASAP.

t1m0thyj
t1m0thyj requested changes Jan 27, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@t1m0thyj t1m0thyj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will approve after zoweVersion is updated 😋

@awharn
Update zoweVersion
3069f04 
Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
@t1m0thyj t1m0thyj removed the Blocked label Jan 27, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jan 27, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@t1m0thyj t1m0thyj merged commit 3f8075b into master Jan 27, 2026
27 checks passed
@t1m0thyj t1m0thyj deleted the update-diff branch January 27, 2026 18:43
@github-project-automation github-project-automation Bot moved this from Review/QA to Closed in Zowe CLI Squad Jan 27, 2026
@t1m0thyj t1m0thyj added release-patch Indicates a patch to existing code has been applied and removed release-patch Indicates a patch to existing code has been applied labels Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@t1m0thyj t1m0thyj t1m0thyj approved these changes

@gejohnston gejohnston gejohnston approved these changes

@zFernand0 zFernand0 zFernand0 approved these changes

@pujal0909 pujal0909 pujal0909 approved these changes

@traeok traeok traeok approved these changes

@anaxceron anaxceron anaxceron approved these changes

@jace-roell jace-roell Awaiting requested review from jace-roell

@ATorrise ATorrise Awaiting requested review from ATorrise

Assignees

@awharn awharn

Labels

None yet

Projects

Zowe CLI Squad
Status: Closed

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@awharn @traeok @t1m0thyj @gejohnston @zFernand0 @pujal0909 @anaxceron @JTonda @zowe-robot