Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTML Reporter: Add basic ES6 Map fallback for fuzzysort.js #1471

Merged
merged 1 commit into from
Aug 16, 2020
Merged

Conversation

Krinkle
Copy link
Member

@Krinkle Krinkle commented Aug 16, 2020

Follows-up 474a708, which added fuzzysort.js, which has an
undocumented dependency on ES6 Map (whilst publishing its code
in UMD/ES5-syntax).

Restore IE 10 compat by adding a simple inline fallback
(it only needs to work with strings).

Also:

  • Restore ESLint settings on test/main/ to explicitly disable
    es6. This used to be the default (we enable es6:true on /src/
    and /test/cli), but now that ESLint enables it by default we need
    to disable it to flag its use.
  • Fix the handful of syntaxes that were introduced and caused
    tests to fail as a result.

Ref #1440.

Follows-up 474a708, which added fuzzysort.js, which has an
undocumented dependency on ES6 Map (whilst publishing its code
in UMD/ES5-syntax).

Restore IE 10 compat by adding a simple inline fallback
(it only needs to work with strings).

Also:
* Restore ESLint settings on test/main/ to explicitly disable
  es6. This used to be the default (we enable es6:true on /src/
  and /test/cli), but now that ESLint enables it by default we need
  to disable it to flag its use.
* Fix the handful of syntaxes that were introduced and caused
  tests to fail as a result.

Ref #1440.
@Krinkle Krinkle merged commit 2f2eaba into master Aug 16, 2020
@Krinkle Krinkle deleted the fix-ie10 branch August 16, 2020 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant