zowe / zowe-cli

Zowe CLI
Eclipse Public License 2.0
114 stars 86 forks source link

v4: Make the `ImperativeError.errorCode` property a `number` #1869

Open zFernand0 opened 1 year ago

zFernand0 commented 1 year ago

Moving the errorCode from a string to a number in the ImperativeError class can bring several benefits. Firstly, using a number for error codes can help reduce the amount of memory used, as numbers take up less space than strings. Secondly, it can improve code efficiency by enabling faster comparisons. Additionally, using a number can also make it easier to integrate with other systems and programming languages, as numbers are a more universally recognized data type. Finally, it can improve the overall clarity of the code, as using numbers for error codes can make the code more concise and easier to read.

Source: https://github.com/zowe/imperative/blob/30e580a4f539a338fc60420f9660749bafc3d131/packages/error/src/ImperativeError.ts#L89-L91

github-actions[bot] commented 5 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

github-actions[bot] commented 5 months ago

This issue has been automatically closed due to lack of activity. In an effort to reduce noise, please do not comment any further.

adam-wolfe commented 5 months ago

Consider revisiting when planning for V4.