Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade ESLint configuration #3463

Closed
2 tasks done
Tracked by #2621 ...
romaricpascal opened this issue Mar 31, 2023 · 0 comments · Fixed by #3770
Closed
2 tasks done
Tracked by #2621 ...

Upgrade ESLint configuration #3463

romaricpascal opened this issue Mar 31, 2023 · 0 comments · Fixed by #3770

Comments

@romaricpascal
Copy link
Member

romaricpascal commented Mar 31, 2023

What

Update our ESLint config to reflect the way that we write JavaScript now that we no longer support IE8-11.

For example, re-enable the no-var rule and update the rules for the es-x plugin.

Fix any linting errors that occur. For example, replace var with let or const as appropriate.

## Why

With our new strategy for browser support and us dropping IE, we can write more modern JavaScript

Who needs to work on this

Developers

Who needs to review this

Developers

Done when

  • ESLint configuration has been updated to match how we write JavaScript
  • JavaScript errors have been fixed
@romaricpascal romaricpascal changed the title Upgrade ESLint Upgrade ESLint configuration Mar 31, 2023
@romaricpascal romaricpascal added this to the v5.0 milestone Mar 31, 2023
@romaricpascal romaricpascal moved this from Backlog 🗄 to Sprint Backlog 🏃🏼‍♀️ in GOV.UK Design System cycle board May 30, 2023
@colinrotherham colinrotherham moved this from Sprint Backlog 🏃🏼‍♀️ to In progress 📝 in GOV.UK Design System cycle board Jun 8, 2023
@colinrotherham colinrotherham self-assigned this Jun 8, 2023
@colinrotherham colinrotherham moved this from In progress 📝 to Done 🏁 in GOV.UK Design System cycle board Jun 8, 2023
@colinrotherham colinrotherham removed their assignment Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants