Skip to content

Conversation

@dxvladislavvolkov
Copy link
Contributor

Remove because codelyzer is deprecated and angular doesn't use the package

@dxvladislavvolkov dxvladislavvolkov self-assigned this Dec 12, 2025
@dxvladislavvolkov dxvladislavvolkov added the dependencies Pull requests that update a dependency file label Dec 12, 2025
Copy link

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 deprecated codelyzer package from the project's dependencies. Codelyzer was a linting tool for Angular applications that worked with TSLint, which has been deprecated in favor of ESLint.

Key Changes:

  • Removed codelyzer package from devDependencies
  • Cleaned up all transitive dependencies of codelyzer from package-lock.json

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Removed codelyzer from devDependencies
package-lock.json Removed codelyzer and all its transitive dependencies (app-root-path, aria-query, axobject-query, ast-types-flow, builtin-modules, css-selector-tokenizer, cssauron, damerau-levenshtein, esprima, fastparse, js-yaml, semver-dsl, sprintf-js, through, and tslint peer dependencies)

The changes are clean and complete. The project already uses ESLint for linting (as evidenced by the .eslintrc.yml configuration and the lint scripts in package.json), so removing codelyzer does not impact the project's linting capabilities. All references to codelyzer have been successfully removed from the dependency tree.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dxvladislavvolkov dxvladislavvolkov merged commit 6492855 into master Dec 12, 2025
10 checks passed
@dxvladislavvolkov dxvladislavvolkov deleted the remove-codelyzer branch December 12, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants