Skip to content

remove super-linter#283

Merged
bdehamer merged 1 commit into
mainfrom
bdehamer/linter-refactor
Aug 26, 2025
Merged

remove super-linter#283
bdehamer merged 1 commit into
mainfrom
bdehamer/linter-refactor

Conversation

@bdehamer

@bdehamer bdehamer commented Aug 26, 2025

Copy link
Copy Markdown
Collaborator

Removes the super-linter action in favor of just running eslint against the source (which is already included in the ci workflow).

Copilot AI review requested due to automatic review settings August 26, 2025 22:41
@bdehamer
bdehamer requested a review from a team as a code owner August 26, 2025 22:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the super-linter GitHub Action and simplifies the linting setup to use ESLint directly. The change streamlines the CI/CD process by eliminating the heavyweight super-linter in favor of running ESLint against the TypeScript source code.

  • Removes the entire super-linter workflow configuration
  • Creates a dedicated TypeScript configuration file for linting
  • Updates ESLint configuration to use the new TypeScript config and ignore itself

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/linter.yml Completely removes the super-linter workflow
.github/linters/tsconfig.json Removes the old TypeScript config used by super-linter
tsconfig.lint.json Adds new dedicated TypeScript configuration for linting
package.json Simplifies lint scripts to use default ESLint config
eslint.config.mjs Updates to use new TypeScript config and ignore ESLint config file

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread package.json
Signed-off-by: Brian DeHamer <[email protected]>
@bdehamer
bdehamer force-pushed the bdehamer/linter-refactor branch from 84dbaff to 6349ee9 Compare August 26, 2025 22:42
@bdehamer
bdehamer merged commit 1e2321d into main Aug 26, 2025
5 checks passed
@bdehamer
bdehamer deleted the bdehamer/linter-refactor branch August 26, 2025 23:05
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.

3 participants