Skip to content

[Issue] Update ESLint configuration #32375

Closed
magento/magento-coding-standard
#347
@m2-assistant

Description

@m2-assistant

This issue is automatically created based on existing pull request: #32343: Update ESLint configuration


Description

The ESLint configuration currently in use was last updated four years ago. A lot has changed in the JavaScript since then. Let's update the configuration to match today's standards.

Related Pull Requests

Fixed Issues

None

Manual testing scenarios

This is a coding standards change and does not make any changes to any functionality within the platform. Therefore no testing of Magento itself is required.

Questions or comments

I expect this pull request to spark some conversation. Many of the settings here have come from the JavaScript Standard Style, although some adaptations have been made to better fit with Magento's existing style guide (eg, four spaces for indentation). I have intentionally kept all existing settings unchanged.

I'm happy to receive feedback here, but may need to get someone from Magento to give a deciding vote / official stance if consensus cannot be amicably achieved.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

Metadata

Metadata

Assignees

No one assigned

    Labels

    Priority: P4No current plan to fix. Fixing can be deferred as a logical part of more important work.Progress: doneTriage: Dev.ExperienceIssue related to Developer Experience and needs help with Triage to Confirm or Reject it

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions