Skip to content

Conversation

@AbhiPrasad
Copy link
Member

Some eslint rules I wanted to add based on Kamil's review comments I saw earlier.

feat(eslint): Add naming convention for typeLike - 6c5cd37

Update @typescript-eslint/naming-convention to use 'PascalCase'
for typeLike declarations -> (class, interface, typeAlias, enum, typeParameter)

feat(eslint): Make class member accessibility explicit - 79be62c

Add @typescript-eslint/explicit-member-accessibility rule to make
sure class members declarations have explicit accessibility (public,
private, protected).

Update @typescript-eslint/naming-convention to use 'PascalCase'
for typeLike declarations -> (class, interface, typeAlias, enum, typeParameter)
@github-actions
Copy link
Contributor

github-actions bot commented May 14, 2021

size-limit report

Path Size
@sentry/browser - CDN Bundle (gzipped) 20.75 KB (-0.01% 🔽)
@sentry/browser - Webpack 21.97 KB (0%)
@sentry/react - Webpack 22 KB (0%)
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped) 28.13 KB (-0.01% 🔽)

@AbhiPrasad AbhiPrasad force-pushed the abhi/eslint-updates branch from 79be62c to ea234b8 Compare May 14, 2021 20:56
Add @typescript-eslint/explicit-member-accessibility rule to make
sure class members declarations have explicit accessibility (public,
private, protected).
@AbhiPrasad AbhiPrasad force-pushed the abhi/eslint-updates branch from ea234b8 to f650a15 Compare May 14, 2021 21:34
@AbhiPrasad AbhiPrasad enabled auto-merge (squash) May 17, 2021 11:13
@AbhiPrasad AbhiPrasad merged commit ac395a0 into master May 17, 2021
@AbhiPrasad AbhiPrasad deleted the abhi/eslint-updates branch May 17, 2021 12:00
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.

3 participants