zowe / zowe-cli

Zowe CLI
Eclipse Public License 2.0
108 stars 85 forks source link

Move Imperative and CLI log files into same folder #2122

Closed t1m0thyj closed 1 month ago

t1m0thyj commented 2 months ago

What It Does

Fixes https://github.com/zowe/zowe-cli/issues/2116

TODO:

How to Test

Build the CLI from this PR and verify that logs are generated in the new location:

Review Checklist I certify that I have:

Additional Comments

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.05%. Comparing base (7272153) to head (a31d320). Report is 2 commits behind head on next.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## next #2122 +/- ## ======================================= Coverage 91.05% 91.05% ======================================= Files 618 618 Lines 17515 17516 +1 Branches 3617 3616 -1 ======================================= + Hits 15948 15949 +1 Misses 1566 1566 Partials 1 1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

gejohnston commented 2 months ago

Should we add something about this in the "end-user functionality" section of the V3 Breaking changes doc? Note that we are now exporting that Word doc to a PDF file in the same directory.

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

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

See analysis details on SonarCloud

github-actions[bot] commented 1 month ago

Release succeeded for the next branch. :tada:

The following packages have been published:

Powered by Octorelease :rocket: