Kristoffer Nordström
Kristoffer Nordström
@SudharakaP It looks like your [eslintconfig](https://github.com/opencollective/opencollective-frontend/blob/main/.eslintrc.json#L2) extends [styled-components-a11y](https://github.com/brendanmorrell/eslint-plugin-styled-components-a11y/blob/master/src/index.js#L44) which extends [eslint-config-airbnb](https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb/rules/react-a11y.js#L123) which force both id & nesting rules to be followed. However in your code you only associated them by...
**What were you expecting to happen?** Expected excluded files in gulp.src to be excluded **What actually happened?** Excluded files with ../ where not excluded on Mac/Linux, seems to work on...
Can also be done with gulp.src cwd option `gulp.src([ 'src/app/scripts/**/*.js', '!src/app/scripts/**/*.spec.js' ], { cwd: '../'});` But I'd prefer not to rewrite all my scripts
Duplicate of #54
Since you state this plugin is meant to be the equivalent to eslint-plugin-jsx-a11y you should probably not extend eslint-config-airbnb/rules/react-a11y since the latter apparently have at least one different rule.
Duplicates #104
Changes detected by argos is as expected
> Please check my comment. @siriwatknp Changes done as requested. Please have a new look 🙏
> Looking good 👌 :+1: @siriwatknp Could you resolve comments & update review from `requested changes` to `approve`? Would have loved to have this for Christmas 🎄