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

Framework: Upgrade ESLint no-nested-ternary to error #9040

Merged
merged 1 commit into from
Nov 1, 2016

Conversation

aduth
Copy link
Contributor

@aduth aduth commented Oct 31, 2016

Related: #8562

This pull request continues the crusade to upgrade all ESLint rules to errors by resolving remaining issues with no-nested-ternary rule.

Testing Instructions:

That CircleCI passes is indication that the changes included are working as intended.

@aduth aduth added Framework [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Type] Janitorial labels Oct 31, 2016
@matticbot
Copy link
Contributor

@aduth aduth force-pushed the update/eslint-no-nested-ternary branch from df10a98 to 0773516 Compare November 1, 2016 18:29
@aduth aduth added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Nov 1, 2016
@aduth aduth merged commit 07fb8a7 into master Nov 1, 2016
@aduth aduth deleted the update/eslint-no-nested-ternary branch November 1, 2016 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants