Skip to content

Commit

Permalink
chore(api): replace locals-rules with @1024pix eslint rules
Browse files Browse the repository at this point in the history
  • Loading branch information
Dimitri Lahaye authored Oct 17, 2023
1 parent c26d121 commit 361cd4f
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 24 deletions.
4 changes: 2 additions & 2 deletions api/.eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ module.exports = {
globals: {
include: true,
},
plugins: ['knex', 'unicorn', 'local-rules'],
plugins: ['knex', 'unicorn', '@1024pix'],
rules: {
'no-console': 'error',
'mocha/no-hooks-for-single-case': 'off',
Expand Down Expand Up @@ -67,6 +67,6 @@ module.exports = {
],
},
],
'local-rules/no-sinon-stub-with-args-oneliner': 'error',
'@1024pix/no-sinon-stub-with-args-oneliner': 'error',
},
};
7 changes: 0 additions & 7 deletions api/eslint-local-rules.cjs

This file was deleted.

17 changes: 5 additions & 12 deletions api/package-lock.json

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

3 changes: 1 addition & 2 deletions api/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@
},
"devDependencies": {
"@1024pix/eslint-config": "^1.0.3",
"@1024pix/eslint-plugin-no-sinon-stub-with-args-oneliner": "^0.2.0",
"@1024pix/eslint-plugin": "^1.0.0",
"@babel/eslint-parser": "^7.18.2",
"@babel/plugin-syntax-import-assertions": "^7.20.0",
"@ls-lint/ls-lint": "^2.0.0",
Expand All @@ -100,7 +100,6 @@
"eslint-plugin-i18n-json": "^4.0.0",
"eslint-plugin-import": "^2.27.5",
"eslint-plugin-knex": "https://github.com/1024pix/eslint-plugin-knex#master",
"eslint-plugin-local-rules": "^2.0.0",
"eslint-plugin-mocha": "^10.0.5",
"eslint-plugin-n": "^16.0.0",
"eslint-plugin-prettier": "^5.0.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ describe('Unit | Scripts | organizations-by-external-id-helper.js', function ()
];

// TODO: Fix this the next time the file is edited.
// eslint-disable-next-line local-rules/no-sinon-stub-with-args-oneliner
// eslint-disable-next-line @1024pix/no-sinon-stub-with-args-oneliner
const findByExternalIdsFetchingIdsOnlyStub = sinon.stub().withArgs(['A100', 'B200']).resolves([]);
const organizationRepository = { findByExternalIdsFetchingIdsOnly: findByExternalIdsFetchingIdsOnlyStub };

Expand Down

0 comments on commit 361cd4f

Please sign in to comment.