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

Remove new-cap eslint-disable wherever possible - Closes #5933 #5967

Merged
merged 3 commits into from Nov 12, 2020

Conversation

ishantiw
Copy link
Member

What was the problem?

This PR resolves #5933

How was it solved?

馃敟 Remove new-cap eslint-disable wherever possible 9bcb274

How was it tested?

  1. Search for eslint-disable-next-line new-cap only one occurrence should be there

    // eslint-disable-next-line new-cap
    const { locale } = Intl.DateTimeFormat().resolvedOptions();

  2. Run yarn lint

@ishantiw ishantiw self-assigned this Nov 11, 2020
@ManuGowda ManuGowda merged commit f231344 into development Nov 12, 2020
@ManuGowda ManuGowda deleted the 5933_use_camelCase_non_cons_var branch November 12, 2020 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename non constructor variables to camelCase
3 participants