Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (531dc7f) to head (d5facae).
Report is 128 commits behind head on master.

@@            Coverage Diff            @@
##            master     #2099   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          104       105    +1     
  Lines         2308      2317    +9     
  Branches       578       578           
=========================================
+ Hits          2308      2317    +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I certainly think these changes would address our concern. We use eslint to help ensure we are staying compliant with older browsers FWIW.

Can you share which rule/plugin you are using? Then I can add those locally to check

.eslintrc.json

{
  "env": {
    "browser": true,
    "es6": false
  },
  "globals": {
    "define": true,
    "msieversion": true
  },
  "plugins": [ "es5" ],
  "extends": [ "plugin:es5/no-es2015" ],
  "parserOptions": { "sourceType": "module" },
  "rules": {
    "no-console": "error",
    "es5/no-block-scoping": "error",
    "es5/no-es6-methods": "warn"
  }
}