k6 icon indicating copy to clipboard operation
k6 copied to clipboard

Drop legacy require support

Open mstoykov opened this issue 10 months ago • 1 comments

What?

Completely drop the legacy require support

Why?

Checklist

  • [ ] I have performed a self-review of my code.
  • [ ] I have added tests for my changes.
  • [ ] I have run linter locally (make lint) and all checks pass.
  • [ ] I have run tests locally (make tests) and all tests pass.
  • [ ] I have commented on my code, particularly in hard-to-understand areas.

Related PR(s)/Issue(s)

mstoykov avatar Apr 19 '24 09:04 mstoykov

Codecov Report

Attention: Patch coverage is 80.95238% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 70.87%. Comparing base (93649df) to head (2961d5a).

:exclamation: Current head 2961d5a differs from pull request most recent head af21b6f

Please upload reports for the commit af21b6f to get more accurate results.

Files Patch % Lines
js/bundle.go 77.77% 2 Missing and 2 partials :warning:
js/modules/resolution.go 82.60% 2 Missing and 2 partials :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3694      +/-   ##
==========================================
- Coverage   70.88%   70.87%   -0.01%     
==========================================
  Files         291      288       -3     
  Lines       21226    21183      -43     
==========================================
- Hits        15045    15013      -32     
+ Misses       5216     5206      -10     
+ Partials      965      964       -1     
Flag Coverage Δ
ubuntu 70.87% <80.95%> (+0.03%) :arrow_up:
windows ?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

codecov-commenter avatar Apr 19 '24 09:04 codecov-commenter

This is now part of #3456

mstoykov avatar Jul 01 '24 09:07 mstoykov