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.

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

Remove deprecated/unstable Utils functions #1947

Closed
1 task
Tracked by #2070
zFernand0 opened this issue May 14, 2020 · 3 comments
Closed
1 task
Tracked by #2070

Remove deprecated/unstable Utils functions #1947

zFernand0 opened this issue May 14, 2020 · 3 comments
Labels
good first issue Good for newcomers priority-low Legit issue but cosmetic or nice-to-have

Comments

@zFernand0
Copy link
Member

In various places we use isNullOrUndefined and it has been deprecated for quite a while (Node v4.0.0)
Maybe we should go back and address some of these in the future?

Items deprecated that we still use:

P.S. Feel free to update the list

@zFernand0 zFernand0 added the priority-low Legit issue but cosmetic or nice-to-have label Nov 17, 2022
@zFernand0
Copy link
Member Author

There as still ~40 instances of isNullOrUndefined
https://github.com/search?q=repo%3Azowe%2Fimperative%20isNullOrUndefined%20&type=code

@gejohnston gejohnston added the good first issue Good for newcomers label Dec 13, 2022
@awharn awharn transferred this issue from zowe/imperative Nov 13, 2023
@adam-wolfe adam-wolfe mentioned this issue Feb 26, 2024
23 tasks
@JTonda
Copy link

JTonda commented Apr 24, 2024

possibly add eslint-deprecation rule
timeboxing to a 3

@awharn
Copy link
Member

awharn commented Jun 17, 2024

Completed with #2169

@awharn awharn closed this as completed Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers priority-low Legit issue but cosmetic or nice-to-have
Projects
None yet
Development

No branches or pull requests

4 participants