chore: [#3518] Enable 'no-public' accessibility rule in the .eslintrc file #4227
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #3518
#minor
Description
This PR removes the current
@typescript-eslint/explicit-member-accessibility
override and adds a new one to the rules' config, throwing an error when declaring apublic
access modifier (ignored forTypescript Constructor Shorthands
(link)).Specific Changes
@typescript-eslint/explicit-member-accessibility
override from the .eslintrc.json file.@typescript-eslint/explicit-member-accessibility
config under the rules object, with the accessibility to 'no-public', and ignoring cases where it's declared in constructors.