fix(eslint): provide default ignores when adding @o3r/eslint-config

AmadeusITGroup · Jan 8, 2025 · af41f0c · af41f0c
1 parent 611b5b9
commit af41f0c
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/packages/@o3r/eslint-config/schematics/ng-add/eslint/index.spec.ts b/packages/@o3r/eslint-config/schematics/ng-add/eslint/index.spec.ts
@@ -63,6 +63,7 @@ describe('update eslint config', () => {
     expect(tree.readText('eslint.local.config.mjs')).toContain(`${monorepoPkgName}/projects`);
     expect(tree.readText('eslint.shared.config.mjs')).toContain(`${monorepoPkgName}/report-unused-disable-directives`);
     expect(tree.readText('eslint.shared.config.mjs')).toContain(`${monorepoPkgName}/eslint-config`);
+    expect(tree.readText('eslint.shared.config.mjs')).toContain(`${monorepoPkgName}/ignores`);
   });
 
   it('should add an eslint config on an application', async () => {
@@ -81,6 +82,7 @@ describe('update eslint config', () => {
     expect(tree.exists(`${libRoot}/tsconfig.eslint.json`)).toBeFalsy();
     expect(tree.readText(`${appRoot}/eslint.config.mjs`)).toContain('import shared from \'../../eslint.shared.config.mjs\'');
     expect(tree.readText(`${appRoot}/eslint.local.config.mjs`)).toContain(`${pckName}/projects`);
+    expect(tree.readText(`${appRoot}/eslint.local.config.mjs`)).toContain(`${pckName}/ignores`);
     expect(tree.readText(`${appRoot}/eslint.local.config.mjs`)).toContain('...globals.browser');
     expect(tree.readJson('angular.json')).toEqual({
       ...angularJsonContent,
@@ -115,6 +117,7 @@ describe('update eslint config', () => {
     expect(tree.exists(`${appRoot}/tsconfig.eslint.json`)).toBeFalsy();
     expect(tree.readText(`${libRoot}/eslint.config.mjs`)).toContain('import shared from \'../../eslint.shared.config.mjs\'');
     expect(tree.readText(`${libRoot}/eslint.local.config.mjs`)).toContain(`${pckName}/projects`);
+    expect(tree.readText(`${libRoot}/eslint.local.config.mjs`)).toContain(`${pckName}/ignores`);
     expect(tree.readText(`${libRoot}/eslint.local.config.mjs`)).not.toContain('...globals.browser');
     expect(tree.readJson('angular.json')).toEqual({
       ...angularJsonContent,

diff --git a/...fig/schematics/ng-add/eslint/templates/project/eslint.local.config.__extension__.template b/...fig/schematics/ng-add/eslint/templates/project/eslint.local.config.__extension__.template
@@ -23,5 +23,9 @@ const __dirname = dirname(__filename);
         ...globals.browser
       }<% } %>
     }
+  },
+  {
+    name: '<%= packageName %>/ignores',
+    ignores: []
   }
 ];
diff --git a/.../schematics/ng-add/eslint/templates/workspace/eslint.shared.config.__extension__.template b/.../schematics/ng-add/eslint/templates/workspace/eslint.shared.config.__extension__.template
@@ -13,6 +13,10 @@ const o3rTemplate = require('@o3r/eslint-config/template');
       reportUnusedDisableDirectives: 'error'
     }
   },
+  {
+    name: '<%= packageName %>/ignores',
+    ignores: []
+  },
   {
     name: '<%= packageName %>/settings',
     settings: {