Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove ts-nocheck annotation from source files #2170

Closed
t1m0thyj opened this issue Mar 3, 2023 · 1 comment · Fixed by #2634
Closed

Remove ts-nocheck annotation from source files #2170

t1m0thyj opened this issue Mar 3, 2023 · 1 comment · Fixed by #2634
Assignees
@t1m0thyj
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have Technical Debt Includes Architecture, Code, Testing, Automation debt
Milestone
v2.14.0

Comments

@t1m0thyj
Copy link
Member

t1m0thyj commented Mar 3, 2023

For example: https://github.com/zowe/vscode-extension-for-zowe/blob/main/packages/zowe-explorer/src/job/ZoweJobNode.ts#L12

To conform to best practices and avoid code smells, we should investigate removing this annotation.
@t1m0thyj t1m0thyj added enhancement New feature or request Technical Debt Includes Architecture, Code, Testing, Automation debt priority-low Legit issue but cosmetic or nice-to-have labels Mar 3, 2023
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

Thank you for raising this issue.
The community has 90 days to upvote 👍 the issue.
If it receives 10 upvotes, we will move it to our backlog. If not, we will close it.

@t1m0thyj t1m0thyj self-assigned this Dec 12, 2023
This was linked to pull requests Dec 16, 2023
Allow users to open data sets and USS files in another codepage #2594
Merged
Refactor node constructors and misc code cleanup #2634
Merged
@t1m0thyj t1m0thyj removed a link to a pull request Dec 20, 2023
Allow users to open data sets and USS files in another codepage #2594
Merged
16 tasks
@JillieBeanSim JillieBeanSim added this to the v2.14.0 milestone Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@t1m0thyj t1m0thyj

Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have Technical Debt Includes Architecture, Code, Testing, Automation debt
Projects
Zowe Explorer for VS Code
Status: Closed
Milestone
v2.14.0
Development

Successfully merging a pull request may close this issue.

Refactor node constructors and misc code cleanup zowe/zowe-explorer-vscode
2 participants
@t1m0thyj @JillieBeanSim