Skip to content

offboard from eslint in superlinter #618

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

Merged
merged 1 commit into from
Apr 28, 2025
Merged

offboard from eslint in superlinter #618

merged 1 commit into from
Apr 28, 2025

Conversation

bdehamer
Copy link
Collaborator

Stop running eslint as part of the super-linter action. We've been pinned to an old version of super-linter cause the newer versions are not compatible with our updated eslint config. Until super-linter has support for eslint v9 we should leave this disabled.

Adds an explicit call to eslint to make-up for disabling it in super-linter.

Signed-off-by: Brian DeHamer <bdehamer@github.com>
@Copilot Copilot AI review requested due to automatic review settings April 28, 2025 17:46
@bdehamer bdehamer requested a review from a team as a code owner April 28, 2025 17:46
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR disables eslint execution within the super-linter action due to compatibility issues with eslint v9, and replaces it with an explicit eslint run step.

  • Upgrades super-linter version from v7.2.1 to v7.3.0.
  • Introduces a new environment variable to further customize linter behavior.
  • Adds an explicit step to run eslint via npm.

@bdehamer bdehamer requested a review from ejahnGithub April 28, 2025 17:50
Copy link
Contributor

@ejahnGithub ejahnGithub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bdehamer bdehamer merged commit d82e7cd into main Apr 28, 2025
19 checks passed
@bdehamer bdehamer deleted the bdehamer/linter branch April 28, 2025 18:30
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