americanexpress / eslint-config-amex

✨ This is American Express' eslint config
Apache License 2.0
40 stars 13 forks source link

feat: don't require descriptions for eslint-enable comments #128

Closed 10xLaCroixDrinker closed 1 day ago

10xLaCroixDrinker commented 5 months ago

Builds on #127

Description

Don't require a description for eslint-enable comments

Please make sure that the PR fulfills these requirements

Motivation and Context

Explanation is rarely necessary in addition to what was described in the eslint-disable comment

How Has This Been Tested?

This rule change was used in americanexpress/one-app#1364

Types of changes

github-actions[bot] commented 3 months ago

This pull request is stale because it has been open 30 days with no activity.

oneamexbot commented 1 day ago

:tada: This PR is included in version 16.2.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: