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: Fuzzy search using fuzzysort #1440

Merged
merged 2 commits into from
Jun 4, 2020

Conversation

ventuno
Copy link
Member

@ventuno ventuno commented May 16, 2020

Sample implementation of: #1438 (fuzzysort-based).

Below search results with different keywords (on test/index.html). The examples below use different values of fuzzysort's threshold option.
-10000 is the value used in fuzzysort's documentation: How To Go Fast · Performance Tips section.

Searches with threshold undefined (default)

Search: "assert".
fuzzysort assert

Search: "beforeEach".
fuzzysort beforeEach

Search: "promiseaware".
fuzzysort promiseaware

Search: "script".
fuzzysort script

Searches with threshold -10000

Search: "assert".
fuzzysort assert (threshold -10000)

Search: "beforeEach".
fuzzysort beforeEach (threshold -10000)

Search: "promiseaware".
fuzzysort promiseaware (threshold -10000)

Search: "script".
fuzzysort script (threshold -10000)

@ventuno ventuno force-pushed the ftr-1438-fuzzysort branch from f35939a to 4434e5c Compare May 21, 2020 04:18
@ventuno ventuno changed the title WIP HTML Reporter: Fuzzy search using fuzzysort HTML Reporter: Fuzzy search using fuzzysort Jun 4, 2020
Copy link
Member

@trentmwillis trentmwillis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After discussion in #1441 and #1442, we've decided this is the better implementation at this time.

This implementation looks good. Thanks for trying out various options!

@trentmwillis trentmwillis merged commit 474a708 into qunitjs:master Jun 4, 2020
@ventuno ventuno deleted the ftr-1438-fuzzysort branch June 4, 2020 16:46
@ventuno
Copy link
Member Author

ventuno commented Jun 4, 2020

Thanks @trentmwillis, when is the next release planned?

@trentmwillis
Copy link
Member

We don't have regularly scheduled releases. That said, I can probably cut a new release this weekend assuming no other bugs/features come up that we'd want to wait for. Will likely be a patch release since this shouldn't affect how tests run at all.

@ventuno
Copy link
Member Author

ventuno commented Jun 4, 2020

That's great. Thanks a lot for your help!

ventuno added a commit to ventuno/qunit that referenced this pull request Jun 13, 2020
@ventuno
Copy link
Member Author

ventuno commented Jul 13, 2020

@trentmwillis I see a new release was made, but I don't see this feature in the latest version. Any chance we could get this out?

Krinkle added a commit that referenced this pull request 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.
@Krinkle
Copy link
Member

Krinkle commented Aug 16, 2020

@ventuno I'm running through the release process now, sorry about the delay. Ref #1463.

Krinkle added a commit that referenced this pull request 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.
@ventuno
Copy link
Member Author

ventuno commented Aug 16, 2020

Thanks for the update @Krinkle :-).

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.

3 participants