k6
k6 copied to clipboard
Drop legacy require support
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)
Codecov Report
Attention: Patch coverage is 79.48718%
with 8 lines
in your changes are missing coverage. Please review.
:exclamation: No coverage uploaded for pull request base (
requireWarning@0a37d2d
). Click here to learn what that means.
:exclamation: Current head e5cf758 differs from pull request most recent head 05d8abf. Consider uploading reports for the commit 05d8abf to get more accurate results
Files | Patch % | Lines |
---|---|---|
js/bundle.go | 77.77% | 2 Missing and 2 partials :warning: |
js/modules/resolution.go | 80.00% | 2 Missing and 2 partials :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## requireWarning #3694 +/- ##
=================================================
Coverage ? 73.25%
=================================================
Files ? 275
Lines ? 20340
Branches ? 0
=================================================
Hits ? 14900
Misses ? 4489
Partials ? 951
Flag | Coverage Δ | |
---|---|---|
ubuntu | 73.25% <79.48%> (?) |
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.