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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(jest/no-identical-title): don't show an error for different template literals #239

Merged
merged 1 commit into from
Mar 12, 2019

Conversation

ronami
Copy link
Contributor

@ronami ronami commented Mar 12, 2019

Summary

Hey, since the recent release (with #237), two different template literals with no expressions will show a lint error. This happens because we try to get their value in a way that doesn't work for template literals and get undefined (see here).

This PR handles this use-case and adds a test to it (which would fail the last release).

Thanks 馃檹

@ronami ronami changed the title fix(jest/no-identical-title): errors shown different string literals fix(jest/no-identical-title): don't show an error for different template literals with no expressions Mar 12, 2019
@ronami ronami changed the title fix(jest/no-identical-title): don't show an error for different template literals with no expressions fix(jest/no-identical-title): don't show an error for different template literals Mar 12, 2019
Copy link
Contributor

@ranyitz ranyitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks

Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@SimenB SimenB merged commit f6f6d84 into jest-community:master Mar 12, 2019
@SimenB
Copy link
Member

SimenB commented Mar 12, 2019

馃帀 This PR is included in version 22.3.2 馃帀

The release is available on:

Your semantic-release bot 馃摝馃殌

@ronami ronami deleted the fix-identical-template-literal branch March 12, 2019 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants