eslint-webpack-plugin icon indicating copy to clipboard operation
eslint-webpack-plugin copied to clipboard

perf: reduce filesystem calls

Open ricardogobbosouza opened this issue 10 months ago • 4 comments

Use the contents of the file using module.originalSource or module._source reducing calls to files.

Now we use the function the lintText instead of lintFiles

*Need to add the performance improvements

This PR contains a:

  • [ ] bugfix
  • [ ] new feature
  • [x] code refactor
  • [ ] test update
  • [ ] typo fix
  • [ ] metadata update

Motivation / Use-Case

Resolve #157

Breaking Changes

No

Additional Info

No

ricardogobbosouza avatar Aug 19 '23 15:08 ricardogobbosouza

Codecov Report

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

Comparison is base (19cadbe) 100.00% compared to head (c6378fd) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #229   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines          291       289    -2     
  Branches        81        75    -6     
=========================================
- Hits           291       289    -2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 19 '23 15:08 codecov[bot]

@ricardogobbosouza Friendly ping, do you need help here?

alexander-akait avatar Dec 11 '23 15:12 alexander-akait

I need to create a large repository to test performance

ricardogobbosouza avatar Dec 11 '23 16:12 ricardogobbosouza

@ricardogobbosouza Do we need this now?

alexander-akait avatar Mar 14 '24 11:03 alexander-akait