Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
const config = require('./src/config');

module.exports = {
...config,
};
8 changes: 0 additions & 8 deletions .eslintrc.json

This file was deleted.

15 changes: 10 additions & 5 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,22 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

## [v0.1.3] - 2024-01-19
## [v0.2.0] - 2024-01-23

### Fixed
### Changed

- changelog version links
- `typescript-eslint` plugin not being declared as dependency
- Default export is now JavaScript eslint config.

### Added

- Different configuration files to JavaScript and TypeScript.
- `eslint-plugin-import` as dependency

### Fixed

- changelog version links
- `typescript-eslint` plugin not being declared as dependency

## [v0.1.2] - 2024-01-17

### Fixed
Expand All @@ -33,7 +38,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

- First release

[v0.1.3]: https://www.npmjs.com/package/@dipcode/eslint-config/v/0.1.3
[v0.2.0]: https://www.npmjs.com/package/@dipcode/eslint-config/v/0.2.0
[v0.1.2]: https://www.npmjs.com/package/@dipcode/eslint-config/v/0.1.2
[v0.1.1]: https://www.npmjs.com/package/@dipcode/eslint-config/v/0.1.1
[v0.1.0]: https://www.npmjs.com/package/@dipcode/eslint-config/v/0.1.0
31 changes: 29 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,35 @@

The package provides Dipcode's `.eslintrc` as an extensible shared config.

1. Run `npm install --save-dev @dipcode/eslint-config`
2. Add `"extends": "@dipcode/eslint-config"` to your `.eslintrc.json`
Install the config

```sh
npm install --save-dev @dipcode/eslint-config
```

Then configure eslint to use the configuration.

### For JavaScript projects

`.eslintrc.json`

```json
{
"root": true,
"extends": "@dipcode/eslint-config"
}
```

### For TypeScript projects

`.eslintrc.json`

```json
{
"root": true,
"extends": "@dipcode/eslint-config/typescript"
}
```

## Links

Expand Down
6 changes: 3 additions & 3 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 9 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@dipcode/eslint-config",
"version": "0.1.3",
"version": "0.2.0",
"description": "Shareable ESLint config for Dipcode.",
"main": "./src/config.js",
"scripts": {
Expand Down Expand Up @@ -28,7 +28,14 @@
"url": "https://github.com/dipcode-software/eslint-config-dipcode/issues"
},
"homepage": "https://github.com/dipcode-software/eslint-config-dipcode#readme",
"exports": "./src/config.js",
"exports": {
".": "./src/config.js",
"./typescript": "./src/typescript.js"
},
"files": [
"src/config.js",
"src/typescript.js"
],
"devDependencies": {
"@dipcode/prettier-config": "0.1.0",
"husky": "8.0.3",
Expand Down
39 changes: 6 additions & 33 deletions src/config.js
Original file line number Diff line number Diff line change
@@ -1,53 +1,26 @@
'use strict';

module.exports = {
env: {
browser: true,
es2022: true,
},
env: { browser: true, es2022: true, commonjs: true },
extends: [
'eslint:recommended',
'plugin:@typescript-eslint/recommended',
'plugin:jsdoc/recommended-typescript-error',
'plugin:import/recommended',
'plugin:import/typescript',
'plugin:promise/recommended',
// This must be the last
'plugin:prettier/recommended',
],
plugins: ['@typescript-eslint', 'simple-import-sort', 'jsdoc'],
parser: '@typescript-eslint/parser',
parserOptions: { project: 'tsconfig.json', sourceType: 'module' },
plugins: ['simple-import-sort', 'jsdoc'],
rules: {
'@typescript-eslint/ban-ts-comment': ['error', { 'ts-expect-error': 'allow-with-description' }],
'camelcase': 'error',
'simple-import-sort/imports': 'error',
'simple-import-sort/exports': 'error',
'no-console': ['error', { allow: ['warn', 'error'] }],
'@typescript-eslint/no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
'import/extensions': [
2,
'always',
{
ignorePackages: true,
pattern: {
js: 'never',
ts: 'never',
tsx: 'never',
},
},
],
'import/extensions': [2, 'always', { ignorePackages: true, pattern: { js: 'never' } }],
},
ignorePatterns: ['node_modules'],
settings: {
'import/parsers': {
'@typescript-eslint/parser': ['.ts', '.tsx', '.d.ts'],
},
'import/resolver': {
node: {
extensions: ['.js', '.json', '.ts', '.tsx', '.d.ts'],
},
},
'import/extensions': ['.js', '.mjs', '.ts', '.tsx', '.d.ts'],
'import/resolver': { node: { extensions: ['.js', '.json'] } },
'import/extensions': ['.js', '.mjs'],
'import/external-module-folders': ['node_modules', 'node_modules/@types'],
},
overrides: [{ files: ['*.ts', '*.tsx'], rules: { 'import/named': 'off', 'import/no-unresolved': 'off' } }],
Expand Down
35 changes: 35 additions & 0 deletions src/typescript.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
'use strict';

const base = require('./config');

module.exports = {
env: { ...base.env },
extends: [
'eslint:recommended',
'plugin:@typescript-eslint/recommended',
'plugin:jsdoc/recommended-typescript-error',
'plugin:import/recommended',
'plugin:import/typescript',
'plugin:promise/recommended',
// This must be the last line
'plugin:prettier/recommended',
],
plugins: ['@typescript-eslint', ...base.plugins],
parser: '@typescript-eslint/parser',
parserOptions: { project: './tsconfig.json', sourceType: 'module' },
rules: {
...base.rules,
'@typescript-eslint/ban-ts-comment': ['error', { 'ts-expect-error': 'allow-with-description' }],
'@typescript-eslint/no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
'import/extensions': [2, 'always', { ignorePackages: true, pattern: { js: 'never', ts: 'never', tsx: 'never' } }],
},
settings: {
...base.settings,
'import/parsers': { '@typescript-eslint/parser': ['.ts', '.tsx', '.d.ts'] },
'import/resolver': {
node: { extensions: [...base.settings['import/resolver'].node.extensions, '.ts', '.tsx', '.d.ts'] },
},
'import/extensions': [...base.settings['import/extensions'], '.ts', '.tsx', '.d.ts'],
},
overrides: [{ files: ['*.ts', '*.tsx'], rules: { 'import/named': 'off', 'import/no-unresolved': 'off' } }],
};