Skip to content

chore: Switch to @tony.ganchev/eslint-plugin-header#38

Merged
NathanZlion merged 1 commit intocloudscape-design:mainfrom
tonyganchev:main
Mar 20, 2026
Merged

chore: Switch to @tony.ganchev/eslint-plugin-header#38
NathanZlion merged 1 commit intocloudscape-design:mainfrom
tonyganchev:main

Conversation

@tonyganchev
Copy link
Copy Markdown
Contributor

@tonyganchev tonyganchev commented Mar 16, 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.

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

Looking forward to your feedback.

How has this been tested?

npm run lint
Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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`.

I've sent PR proposals to other projects in the _cloudscape-design_
space:
- cloudscape-design/board-components#401
- cloudscape-design/browser-test-tools#188
- cloudscape-design/chart-components#185
- cloudscape-design/code-view#122
- cloudscape-design/collection-hooks#136
- cloudscape-design/component-toolkit#200
- cloudscape-design/components#4308
- cloudscape-design/demos#243
- cloudscape-design/documenter#112
- cloudscape-design/global-styles#73
- cloudscape-design/jest-preset#58
- cloudscape-design/test-utils#114
- cloudscape-design/theming-core#148

Looking forward to your feedback.
@tonyganchev tonyganchev requested a review from a team as a code owner March 16, 2026 11:05
@tonyganchev tonyganchev requested review from NathanZlion and removed request for a team March 16, 2026 11:05
@NathanZlion NathanZlion changed the title chore: switch to @tony.ganchev/eslint-plugin-header chore: Switch to @tony.ganchev/eslint-plugin-header Mar 18, 2026
@NathanZlion NathanZlion enabled auto-merge March 18, 2026 10:44
@tonyganchev
Copy link
Copy Markdown
Contributor Author

@NathanZlion - the codecov failure should be unrelated to my change. Can you help troubleshoot? I think I saw other failures in different linked PRs.

Thanks!

@NathanZlion NathanZlion added this pull request to the merge queue Mar 20, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 20, 2026
@NathanZlion NathanZlion added this pull request to the merge queue Mar 20, 2026
Merged via the queue into cloudscape-design:main with commit 513f4b1 Mar 20, 2026
122 of 126 checks passed
@tonyganchev
Copy link
Copy Markdown
Contributor Author

ecosystem test for this and other cloudscape project: tonyganchev/eslint-plugin-header#163

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