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: Quick fix all for JS+TS #42358

Closed
3 tasks done
mjbvz opened this issue Jan 29, 2018 · 0 comments
Closed
3 tasks done

Test: Quick fix all for JS+TS #42358

mjbvz opened this issue Jan 29, 2018 · 0 comments

Comments

@mjbvz
Copy link
Collaborator

mjbvz commented Jan 29, 2018

#40170

Complexity: 2

OS:

Summary

TS can now apply some quick fixes to all applicable locations in a file. Example fixallable declarations include: removing unused declarations and adding this.

Testing

  1. In a TS file

  2. Introduce a quick fixable error (such as a missing this. for a member references)

  3. Introduce a second error of the same type in the file

    • ✅ Normal quick fix shows up for each error plus a quick fix all action
    • ✅ Quick fix all action fixes both errors
  4. Try introducing other errors in the file and quick fixing them

    • ✅ Do quick fix all actions show up when they make?
    • ✅ Do they do the right thing when you apply them?
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

5 participants