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

Test organize imports #46589

Closed
2 tasks done
mjbvz opened this issue Mar 26, 2018 · 0 comments
Closed
2 tasks done

Test organize imports #46589

mjbvz opened this issue Mar 26, 2018 · 0 comments

Comments

@mjbvz
Copy link
Contributor

mjbvz commented Mar 26, 2018

Complexity: 2

Overview

The new organize imports command for JS and TS removes unused imports and sorts the remaining imports. It can be triggered using a command or with alt+shift+o

Testing

  • Try using organize imports in the vscode codebase
    • Try introducing some unused imports and verifying that these are removed
  • Try using organize imports a create react app javascript project
  • Try using organize imports in an angular typescript project

Important Note

I've already reported a few import bugs around organize imports to TS such as microsoft/TypeScript#22731. If you find a bug, please install typescript@next in your workspace and check if the bug still repos using the latest typescript version. Many of important organize imports bugs have already been fixed upstream but unfortunately there is nothing we can do on the VS Code side for them at the moment @

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants