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

Deterministic sorting of search results #62

Merged
merged 4 commits into from
Jun 30, 2019

Conversation

xaviershay
Copy link

This is important for reliable tests, e.g. if you are snapshotting views and comparing to reference images.

This is important for reliable tests, e.g. if you are snapshotting views and comparing to reference images.
@johnsenpeder johnsenpeder self-assigned this May 9, 2019
@pederjohnsen
Copy link
Member

Hey @xaviershay!

Thanks for this PR, however when it comes to the core code shared with https://github.com/missive/emoji-mart I prefer porting changes from there as improvements are added. Could you suggest your PR there first? :)

@xaviershay
Copy link
Author

here you go! missive/emoji-mart#343

@johnsenpeder
Copy link
Contributor

@xaviershay great thanks 👍

@johnsenpeder
Copy link
Contributor

@xaviershay sorry, just closed and re-opened this to force a travis ci build. You'll need to run yarn prettier for this PR and the emoji-mart one to fix the errors :)

@pederjohnsen pederjohnsen merged commit 2856582 into tunoltd:master Jun 30, 2019
@johnsenpeder
Copy link
Contributor

@xaviershay just published new version to npm with these changes 👌 🚀

@xaviershay
Copy link
Author

love your work ❤️ 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants