faker icon indicating copy to clipboard operation
faker copied to clipboard

infra: update eslint config

Open Shinigami92 opened this issue 5 months ago • 3 comments

While configuring projects at company work, I copied over some huge parts of Faker's structure into the company projects. While doing that, I found out that some parts are not working as intended or outdated.

This PR fixes some issues with out current eslint.config.ts

Shinigami92 avatar Jun 15 '25 07:06 Shinigami92

Deploy Preview for fakerjs ready!

Name Link
Latest commit 2b578aac7d1b465eb02d668767cf29fce49469fc
Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/6853eebc50c62b00080cae82
Deploy Preview https://deploy-preview-3530.fakerjs.dev
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar Jun 15 '25 07:06 netlify[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.97%. Comparing base (b357ea0) to head (2b578aa). Report is 1 commits behind head on next.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3530   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files        2880     2880           
  Lines      220510   220510           
  Branches      952      952           
=======================================
  Hits       220457   220457           
  Misses         53       53           
:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jun 15 '25 07:06 codecov[bot]

Only one comment. The rest looks fine to me. 👍

xDivisionByZerox avatar Jun 15 '25 10:06 xDivisionByZerox

Since this PR is Infrastructure related, I'd say we can merge this without a second review before the usual 7 day limit. Dependency PR are handled in a similar manner.

xDivisionByZerox avatar Jun 19 '25 11:06 xDivisionByZerox

I'm fine with that

Shinigami92 avatar Jun 19 '25 11:06 Shinigami92