Closed t1m0thyj closed 2 months ago
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 93.40%. Comparing base (
cb6aba7
) to head (de7b9dc
). Report is 1 commits behind head on next.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I don't think we have an open issue for addressing scan results in general, but I think this is a good practice to continue doing.
@t1m0thyj, Should we open an issue about the settingEnabled
bug that was theoretically fixed here? Not sure if we really need it, but it might be good to capture it somewhere 😋
merging with the approvals. also for follow up @t1m0thyj, here is the issue I mentioned in scrum that may be related https://github.com/zowe/zowe-explorer-vscode/issues/2063
Proposed changes
Fixes 4 code smells detected by Coverity in the v3 branch
Release Notes
Milestone:
Changelog:
Types of changes
What types of changes does your code introduce to Zowe Explorer? Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This checklist will be used as reference for both the contributor and the revieweryarn workspace vscode-extension-for-zowe vscode:prepublish
has been executedFurther comments