-
Notifications
You must be signed in to change notification settings - Fork 228
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
Fix false positives with no-jasmine-globals #205
Conversation
Not that exciting right now, but the upcoming commit needs the grouped if-block anyway.
Want to re-use it for no-jasmine-globals
Travis is upset about my commit message:
Is that because of the capital "O" in spyOn? [Update: Oh, it's the capital F in Fix] |
Prior to this, similarly named local variables & parameters would trigger the no-jasmine-globals warning Fixes jest-community#156
a3dd6c2
to
7f7b5ed
Compare
This is great, thanks! I left a nit, but would love for @macklinu to chime in as well before merging 🙂 |
@macklinu could you take a look when you find the time? Feel free to merge (squash merge, make sure the commit message is semantic) if it's OK 🙂 |
🎉 This PR is included in version 22.0.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@SimenB sorry I didn’t get a chance to look before merging. Have been away from the computer for Thanksgiving holiday. This change looks great though! 🎉 |
See #156. WDYT?