Skip to content

chore: switch to @tony.ganchev/eslint-plugin-header#200

Merged
georgylobko merged 1 commit intocloudscape-design:mainfrom
tonyganchev:main
Mar 27, 2026
Merged

chore: switch to @tony.ganchev/eslint-plugin-header#200
georgylobko merged 1 commit intocloudscape-design:mainfrom
tonyganchev:main

Conversation

@tonyganchev
Copy link
Copy Markdown
Contributor

@tonyganchev tonyganchev commented Mar 15, 2026

Hi, team,

I noticed you are using eslint-plugin-header on ESLint 8.

I forked @tony.ganchev/eslint-plugin-header mid-2024 to add support
for ESLint 9 and hoped it would be a temporary measure but since the
original has not been updated for five years I decided to continue
improving the new plugin and have been doing so for the last two years.

Specific improvements include:

  • full JSON schema for validating the configuration.
  • fixed multiple bugs with the behavior of the plugin on Windows.
  • many other bug-fixes.
  • improved autofixing and error-reporting behavior.
  • added support for leading pragma comments before the header such as
    @jest-environment. In the case of the current review, this allowed
    me to remove the eslint-disable directive from two tests and
    reintroduce validation which was actually failing because of the
    missing empty line after the header.

I switched the configuration to use the new config format for v3.2.x
as the leading comments feature is not supported by the array-based
format inherited from eslint-plugin-header.

I've sent PR proposals to other projects in the cloudscape-design
space:

Looking forward to your feedback.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants