Skip to content

Conversation

bdehamer
Copy link
Collaborator

@bdehamer bdehamer commented Aug 26, 2025

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

@Copilot 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
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 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.

Signed-off-by: Brian DeHamer <bdehamer@github.com>
@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.

2 participants