vscode-nuget-package-manager icon indicating copy to clipboard operation
vscode-nuget-package-manager copied to clipboard

General code cleanup task

Open jmrog opened this issue 8 years ago • 2 comments

In some cases, functions return with a large block wrapped in new Promise((resolve, reject) => { . . . }, when it probably isn't necessary (I'm not sure because I'm basing this on memory alone right now) to wrap the whole block like that. This could be cleaned up. Also consider using async/await.

jmrog avatar Jul 01 '17 15:07 jmrog

Additionally, there are unnecessarily verbose index.ts files used just for exporting. They import first and then export, when they could simply just export directly from the files.

jmrog avatar Nov 18 '17 03:11 jmrog

More: some of the code could use better typing.

jmrog avatar Nov 18 '17 16:11 jmrog