zowe / zowe-cli

Zowe CLI
Eclipse Public License 2.0
113 stars 87 forks source link

Removing profile versioning return value from commands about environment #2053

Closed ATorrise closed 7 months ago

ATorrise commented 7 months ago

What It Does Part of the modernization effort in preparation for v3. Given v1 profiles will no longer be supported, their consideration in certain command logic is no longer necessary.

How to Test

  1. Append your command with --show-inputs-only.
  2. Test zowe config report-env. Ensure neither command returns anything related to differentiating your environment between TeamConfig of v1

Review Checklist I certify that I have:

Additional Comments If there are any other commands the team can think of that return information about environment for user information that have special logic for v1, the removal of that logic could be appended to this PR.

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 40.00000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 88.79%. Comparing base (3fcffd1) to head (ff4fbd4).

:exclamation: Current head ff4fbd4 differs from pull request most recent head ae06f7f. Consider uploading reports for the commit ae06f7f to get more accurate results

Files Patch % Lines
...ackages/imperative/src/cmd/src/CommandProcessor.ts 40.00% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## rm-profile-handlers #2053 +/- ## ======================================================= - Coverage 90.92% 88.79% -2.13% ======================================================= Files 616 616 Lines 17165 17153 -12 Branches 3616 3551 -65 ======================================================= - Hits 15607 15231 -376 - Misses 1557 1921 +364 Partials 1 1 ```

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

zFernand0 commented 7 months ago

Smart to have based your branch off of the rm-profile-handlers branch 🥳 The only downside is that you may run into issues coming from that WIP branch 😋

Changes LGTM! 😋

just gotta address a few failing tests 😅

gejohnston commented 7 months ago

Thanks for going beyond just changing the text that is displayed. The removal of V1 logic in report-env is above and beyond the original issue, but a very worthwhile change.

I confirmed that your branch has the same bug fix as the original branch from which you created your branch. So you were right, your test errors are the result of some specific changes in your branch.

sonarcloud[bot] commented 7 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues

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

See analysis details on SonarCloud

ATorrise commented 7 months ago

@gejohnston Are these failing tests acceptable or should I do something about them?

gejohnston commented 7 months ago

Are these failing tests acceptable or should I do something about them?

@ATorrise I think that we should merge this branch into rm-profile-handlers. I think that the error will not exist in the rm-profile-handlers branch. Even if there is a problem, it will be easier to fix in that branch.