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

feat: expose array-like methods for searching data #4

Merged
merged 4 commits into from
Aug 4, 2020

Conversation

trs
Copy link
Contributor

@trs trs commented Aug 4, 2020

Instead of limiting an end-user to a string search, enable predicate functions to find and filter values.

BREAKING CHANGE: exposed api changed from string search to predicate methods

@trs trs requested a review from matt-forster August 4, 2020 14:57
@trs trs self-assigned this Aug 4, 2020
@trs trs force-pushed the array-like-methods branch from 038c1ca to 88dbf0d Compare August 4, 2020 14:59
@trs trs mentioned this pull request Aug 4, 2020
2 tasks
@trs trs marked this pull request as ready for review August 4, 2020 16:54
@trs trs force-pushed the array-like-methods branch from 88dbf0d to 72cc659 Compare August 4, 2020 17:01
src/index.d.ts Outdated Show resolved Hide resolved
@trs trs force-pushed the array-like-methods branch 3 times, most recently from 3356bb9 to 2464c38 Compare August 4, 2020 17:14
Comment on lines +92 to +93
cityData.name,
cityData.province,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The data was manipulated to make the specific implementation of text search easier. But with that gone, I think it makes sense to expose these as-is.

@trs trs requested a review from matt-forster August 4, 2020 17:15
Tyler Stewart and others added 4 commits August 4, 2020 11:22
Instead of limiting an end-user to a string search, enable `predicate` functions to `find` and `filter` values.

BREAKING CHANGE: exposed api changed from string search to predicate methods
Co-authored-by: forstermatth <matt@autovance.com>
@trs trs force-pushed the array-like-methods branch from 2464c38 to 18ecb5a Compare August 4, 2020 17:22
@@ -1,13 +1,12 @@
{
"name": "canadian-city-timezones",
"version": "1.0.0",
"version": "2.0.0",

Choose a reason for hiding this comment

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

If semver has been set up, do we need to change this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not at all! Could even make it 0.0.0 or something.

@trs trs merged commit 69e0603 into master Aug 4, 2020
@trs trs deleted the array-like-methods branch August 4, 2020 19:20
@github-actions
Copy link

github-actions bot commented Aug 4, 2020

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants