hackforla / CivicTechJobs

CivicTechJobs will be a platform to help prospective volunteers find interdisciplinary projects that will be useful for their career development while contributing to positive civic impact, and also a CMS (Content Mgmt System) for Hack for LA projects to be able to list their open roles.
https://civictechjobs.org
GNU General Public License v2.0
19 stars 20 forks source link

Add linting support #99

Closed Aveline-art closed 2 years ago

Aveline-art commented 2 years ago

Overview

As a developer, we need to periodically review our code to make sure that it is up-to-code, up-to-date, and follow best practices. For this issue, we will discuss linters we can use and install them.

Action Items

Resources/Instructions

Lint (software) Eslint Prettier About continuous integration GitHub Actions github/super-linter

Before working on this issue, be aware that GitHub incorporates its own way of doing CD/CI, via GitHub Actions (GHA). Typically what happens is that a pull request would be created, a GHA is then triggered to generate a linting report to inform the author of the pull request. This report list out linting flags to alert the author and assist them in updating areas of their code. As a matter of fact there are several pre-made GHAs that performs linting. However, linting can encompass many things; it is, after all, a form of code testing. Therefore, for this issue, do broad research on linters so that the team can decide which one best fits our needs.

Aveline-art commented 2 years ago

Questions for all team:

The pluggable linting tool for text and markdown. textlint is similar to ESLint, but it's for use with natural language.

Questions for dev:

  1. To Prettier or not to Prettier
Aveline-art commented 2 years ago

After discussion with the team:

  1. fallback font will be the arial family
  2. Textlint will be used for our md files.
  3. Prettier will be our standard formatter for all code that it supports