1.3.1 • Published 5 months ago

@sitek94/style-guide v1.3.1

Weekly downloads
-
License
MPL-2.0
Repository
github
Last release
5 months ago

Maciek's Style Guide

npm version

I want to keep consistent code style across all my projects. Initially, this repository is a clone of Vercel's Style Guide and I will be modifying it to my needs.

The following configs are available, and are designed to be used together.

Installation

All configs are contained in one package, @sitek94/style-guide. To install:

npm i -D @sitek94/style-guide

pnpm i -D @sitek94/style-guide

yarn add -D @sitek94/style-guide

bun add -D @sitek94/style-guide

Some of our ESLint configs require peer dependencies. We'll note those alongside the available configs in the ESLint section.

Prettier

Note: Prettier is a peer-dependency of this package, and should be installed at the root of your project.

See: Prettier: Getting Started

To use the shared Prettier config, set the following in package.json.

{
  "prettier": "@sitek94/style-guide/prettier"
}

ESLint

Note: ESLint is a peer-dependency of this package, and should be installed at the root of your project.

See: ESLint: Getting Started

This ESLint config is designed to be composable.

The following base configs are available. You can use one or both of these configs, but they should always be first in extends:

  • @vercel/style-guide/eslint/browser
  • @vercel/style-guide/eslint/node

Note that you can scope configs, so that configs only target specific files. For more information, see: Scoped configuration with overrides.

The following additional configs are available:

  • @vercel/style-guide/eslint/jest
  • @vercel/style-guide/eslint/jest-react (includes rules for @testing-library/react)
  • @vercel/style-guide/eslint/next (requires @next/eslint-plugin-next to be installed at the same version as next)
  • @vercel/style-guide/eslint/playwright-test
  • @vercel/style-guide/eslint/react
  • @vercel/style-guide/eslint/typescript (requires typescript to be installed and additional configuration)

You'll need to use require.resolve to provide ESLint with absolute paths, due to an issue around ESLint config resolution (see eslint/eslint#9188).

For example, use the shared ESLint config(s) in a Next.js project, set the following in .eslintrc.js.

module.exports = {
  extends: [
    require.resolve('@vercel/style-guide/eslint/browser'),
    require.resolve('@vercel/style-guide/eslint/react'),
    require.resolve('@vercel/style-guide/eslint/next'),
  ],
}

Configuring ESLint for TypeScript

Some of the rules enabled in the TypeScript config require additional type information, you'll need to provide the path to your tsconfig.json.

For more information, see: https://typescript-eslint.io/docs/linting/type-linting

const {resolve} = require('path')

const project = resolve(__dirname, 'tsconfig.json')

module.exports = {
  root: true,
  extends: [
    require.resolve('@vercel/style-guide/eslint/node'),
    require.resolve('@vercel/style-guide/eslint/typescript'),
  ],
  parserOptions: {
    project,
  },
  settings: {
    'import/resolver': {
      typescript: {
        project,
      },
    },
  },
}

Configuring custom components for jsx-a11y

It's common practice for React apps to have shared components like Button, which wrap native elements. You can pass this information along to jsx-a11y via the components setting.

The below list is not exhaustive.

module.exports = {
  root: true,
  extends: [require.resolve('@vercel/style-guide/eslint/react')],
  settings: {
    'jsx-a11y': {
      components: {
        Article: 'article',
        Button: 'button',
        Image: 'img',
        Input: 'input',
        Link: 'a',
        Video: 'video',
      },
    },
  },
}

Scoped configuration with overrides

ESLint configs can be scoped to include/exclude specific paths. This ensures that rules don't "leak" into places where those rules don't apply.

In this example, Jest rules are only being applied to files matching Jest's default test match pattern.

module.exports = {
  extends: [require.resolve('@vercel/style-guide/eslint/node')],
  overrides: [
    {
      files: ['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)'],
      extends: [require.resolve('@vercel/style-guide/eslint/jest')],
    },
  ],
}

A note on file extensions

By default, all TypeScript rules are scoped to files ending with .ts and .tsx.

However, when using overrides, file extensions must be included or ESLint will only include .js files.

module.exports = {
  overrides: [{files: [`directory/**/*.[jt]s?(x)`], rules: {'my-rule': 'off'}}],
}

TypeScript

This style guide provides multiple TypeScript configs. These configs correlate to the LTS Node.js versions, providing the appropriate lib, module, target, and moduleResolution settings for each version. The following configs are available:

Node.js VersionTypeScript Config
v16@vercel/style-guide/typescript/node16
v18@vercel/style-guide/typescript/node18
v20@vercel/style-guide/typescript/node20

To use the shared TypeScript config, set the following in tsconfig.json.

{
  "extends": "@vercel/style-guide/typescript/node16"
}

The base TypeScript config is also available as @vercel/style-guide/typescript which only specifies a set of general rules. You should inherit from this file when setting custom lib, module, target, and moduleResolution settings.