Remove unused variables with the Babel plugin#21339
Conversation
817af17 to
4bc4343
Compare
|
Oh well, GitHub actions is down. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #21339 +/- ##
==========================================
- Coverage 81.42% 81.42% -0.01%
==========================================
Files 257 257
Lines 65351 65351
==========================================
- Hits 53210 53209 -1
- Misses 12141 12142 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
4bc4343 to
90f8cdb
Compare
- functions were already removed - variables can be removed when their initializer does not have side effects - classes can be removed when they have no static blocks Fixes mozilla#21337
90f8cdb to
2691868
Compare
|
/botio browsertest |
From: Bot.io (Linux m4)ReceivedCommand cmd_browsertest from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/8358dc000813fef/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_browsertest from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/6297e9a7697e0d8/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/8358dc000813fef/output.txt Total script time: 18.24 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.193.163.58:8877/6297e9a7697e0d8/output.txt Total script time: 25.50 mins
|
|
Despite all tests passing, it seems that this broke the GENERIC viewer since opening https://mozilla.github.io/pdf.js/web/viewer.html now fails: @nicolo-ribaudo Should we revert this for now, to not block other work, or could this be fixed quickly? |
Fixes #21337