baseline icon indicating copy to clipboard operation
baseline copied to clipboard

Introduce a linter and formatting style for the SRI project

Open ognjenkurtic opened this issue 2 years ago • 4 comments

Overview

We need this to make sure formatting is uniform across the project.

Reference

Questions

  • Linter?
  • Style?

Acceptance

  • Linter introduced
  • Formatting style agreed and introduces

Tasks

  • [ ] Linter setup and running locally

ognjenkurtic avatar Aug 02 '22 19:08 ognjenkurtic

I vote for typescript-eslint and a modified version of prettier.

Kasshern avatar Aug 03 '22 17:08 Kasshern

I vote for typescript-eslint and a modified version of prettier.

+1 for typescript-eslint. I think default prettier does pretty good job, is there a reason for the proposed modification?

Manik-Jain avatar Aug 04 '22 01:08 Manik-Jain

Its probably best to go with default options and no (or very minial) config in both linter and prettier for start

skosito avatar Aug 04 '22 12:08 skosito

You make a very good point @skosito. You have convinced me, albeit reluctantly, to adopt default options for both the linter and prettier.

Kasshern avatar Aug 04 '22 14:08 Kasshern