Do we add Prettier and or ESLint and configs?

[gbowne1]

I was just thinking it might be somewhat helpful to keep things clean pre-commit by adding Prettier and or ESLint and their configuration(s) to the project if so, which and what would the configuration look like?

I do have some ideas..

.eslintrc.js

export default {
  root: true,
  env: {
    node: true,
    es2022: true,
  },
  parserOptions: {
    ecmaVersion: 'latest',
    sourceType: 'module',
  },
  extends: [
    'eslint:recommended',
  ],
  rules: {
    // Possible errors
    'no-console': 'off', // Useful for logging in backend apps

    // Best practices
    'eqeqeq': ['warn', 'always'],
    'no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
    'no-async-promise-executor': 'warn',

    // Stylistic preferences (customize as needed)
    'semi': ['warn', 'always'],
    'quotes': ['warn', 'single'],
    'indent': ['warn', 4],
    'comma-dangle': ['warn', 'only-multiline'],
    'object-curly-spacing': ['warn', 'always'],

    // Node.js specific
    'no-path-concat': 'error'
  }
};

.prettierrc.js

export default {
    semi: true,
    singleQuote: true,
    trailingComma: 'es5',
    tabWidth: 4,
    printWidth: 100,
    bracketSpacing: true,
    arrowParens: 'avoid',
};

[gbowne1]

@rconjoe @iAmNsengi @itswael @rconjoe @IGIRANEZAJosue @ZennilGreenherald @siferot @Nicode611

[gbowne1]

I would also consider adding:

'no-implicit-globals': 'error',
'no-undef': 'error',
"no-var": "error",
'no-duplicate-imports': 'error'
'prefer-const': 'error',
'arrow-spacing': ['error', { before: true, after: true }],

[josue-igiraneza]

@gbowne1, I agree with the recommendations, and I think it would definitely help maintain quality and consistency

[gbowne1]

Yes @IGIRANEZAJosue I agree

I'm just against adding lint and prettier rules we havent discussed yet to make sure we are all on the same page.

I'm ok with adding whatever rules will follow the style we choose.

We can then run lint and prettier as a whole.

I don't want too much enforcement because I'm all about developer freedom but not too little either.

I did use my local lint rules to lint and prettier during the initial phase.