Skip to content

ESLint Config Migration: Enum members should use PascalCase#1052

Merged
filmaj merged 1 commit intotslint-to-eslintfrom
enum-members-pascal-case
Aug 23, 2021
Merged

ESLint Config Migration: Enum members should use PascalCase#1052
filmaj merged 1 commit intotslint-to-eslintfrom
enum-members-pascal-case

Conversation

@filmaj
Copy link
Copy Markdown
Contributor

@filmaj filmaj commented Aug 9, 2021

Summary

This is a PR that should be merged into #1024 and incrementally addresses #842.

It enforces a naming convention around Enum Members to be PascalCase.

Impact

Before

✖ 362 problems (173 errors, 189 warnings)
  76 errors and 0 warnings potentially fixable with the `--fix` option.

After

✖ 346 problems (157 errors, 189 warnings)
  76 errors and 0 warnings potentially fixable with the `--fix` option.

Requirements (place an x in each [ ])

@filmaj filmaj added the tests M-T: Testing work only label Aug 9, 2021
@filmaj filmaj self-assigned this Aug 9, 2021
@filmaj filmaj changed the title Enum members should use PascalCase. ESLint Config Migration: Enum members should use PascalCase Aug 9, 2021
@filmaj filmaj force-pushed the enum-members-pascal-case branch from 804469e to 38080d2 Compare August 10, 2021 00:35
@filmaj filmaj force-pushed the enum-members-pascal-case branch from 38080d2 to 001b283 Compare August 23, 2021 17:13
@filmaj filmaj merged commit 84afc50 into tslint-to-eslint Aug 23, 2021
@filmaj filmaj deleted the enum-members-pascal-case branch August 23, 2021 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests M-T: Testing work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants