Merge pull request #1200 from priyang12/master

Modify .eslintrc.cjs for explicit-member-accessibility
mathesar-foundation · Mar 22, 2022 · 4be6ffe · 4be6ffe
2 parents 37c32d0 + 26b7b24
commit 4be6ffe
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/mathesar_ui/.eslintrc.cjs b/mathesar_ui/.eslintrc.cjs
@@ -21,6 +21,7 @@ module.exports = {
   rules: {
     'import/no-extraneous-dependencies': ['error', { devDependencies: true }],
     'no-console': ['warn', { allow: ['error'] }],
+    '@typescript-eslint/explicit-member-accessibility': 'off',
     '@typescript-eslint/ban-ts-comment': [
       'error',
       {
@@ -106,6 +107,15 @@ module.exports = {
         '@typescript-eslint/require-await': 'off',
       },
     },
+    {
+      files: ['*.ts', '*.tsx'],
+      rules: {
+        '@typescript-eslint/explicit-member-accessibility': [
+          'error',
+          { accessibility: 'no-public' },
+        ],
+      },
+    },
   ],
   env: {
     es6: true,

diff --git a/mathesar_ui/src/component-library/common/utils/contextValidationUtil.ts b/mathesar_ui/src/component-library/common/utils/contextValidationUtil.ts
@@ -8,11 +8,11 @@ export type ValidationResultStore = Writable<boolean>;
 const VALIDATION_CONTEXT_KEY = 'validationContext';
 
 class ContextBasedValidator {
-  public validationResult: Writable<boolean> = writable(true);
+  validationResult: Writable<boolean> = writable(true);
 
-  public validationFunctionMap: Map<string, ValidationFunction> = new Map();
+  validationFunctionMap: Map<string, ValidationFunction> = new Map();
 
-  public validate(): boolean {
+  validate(): boolean {
     let isValid = true;
     // eslint-disable-next-line no-restricted-syntax
     for (const validationFn of this.validationFunctionMap.values()) {
@@ -22,7 +22,7 @@ class ContextBasedValidator {
     return isValid;
   }
 
-  public addValidator(key: string, fn: ValidationFunction) {
+  addValidator(key: string, fn: ValidationFunction) {
     this.validationFunctionMap.set(key, fn);
 
     onDestroy(() => {