Skip to content

chore(deps): updated eslint#19

Merged
CorentinTh merged 2 commits into
mainfrom
update-eslint
Sep 20, 2025
Merged

chore(deps): updated eslint#19
CorentinTh merged 2 commits into
mainfrom
update-eslint

Conversation

@CorentinTh

Copy link
Copy Markdown
Owner

No description provided.

@CorentinTh CorentinTh requested a review from Copilot September 20, 2025 23:46
@CorentinTh CorentinTh self-assigned this Sep 20, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

Updates ESLint and its configuration to newer versions and refactors code to align with updated linting rules.

  • Updated ESLint from v9.32.0 to v9.36.0 and @antfu/eslint-config from v5.0.0 to v5.4.1
  • Added TypeScript configuration path to ESLint config
  • Refactored code to meet updated linting standards, including condition improvements and function extraction

Reviewed Changes

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

File Description
package.json Updated ESLint and @antfu/eslint-config to newer versions
eslint.config.js Added TypeScript configuration path for better linting
src/utils.ts Improved condition check for undefined values in mergeDeep function
src/figue.errors.ts Extracted issue serialization logic into separate function and improved array handling
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

Comment thread src/figue.errors.ts
@CorentinTh CorentinTh enabled auto-merge (squash) September 20, 2025 23:49
@CorentinTh CorentinTh merged commit 78e6d48 into main Sep 20, 2025
2 checks passed
@CorentinTh CorentinTh deleted the update-eslint branch September 20, 2025 23:49
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