Skip to content
New issue

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

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

Already on GitHub? # to your account

New Function filterBy #10

Merged
merged 3 commits into from
Sep 19, 2018
Merged

New Function filterBy #10

merged 3 commits into from
Sep 19, 2018

Conversation

gustafsilva
Copy link
Contributor

Function creation and testing for filterBy. #6

@coveralls
Copy link

Pull Request Test Coverage Report for Build 119

  • 9 of 9 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 115: 0.0%
Covered Lines: 223
Relevant Lines: 223

💛 - Coveralls

@cahilfoley cahilfoley changed the base branch from master to develop September 19, 2018 11:40
@cahilfoley cahilfoley merged commit ab80d61 into cahilfoley:develop Sep 19, 2018
@cahilfoley
Copy link
Owner

You're a champion! Thanks for this, looks great 👍 🥇

@gustafsilva
Copy link
Contributor Author

You're a champion! Thanks for this, looks great +1 1st_place_medal

Thanks you, i'm learning a lot how to contribute with open source. 😆
Thanks for the opportunity... 👍

cahilfoley added a commit that referenced this pull request Sep 21, 2018
* feat(array): add `filterBy` function (#10)

* chore: update `babel` dependencies

* dev: add `yarn` to the `devDependencies`

fixes issue #9

* docs: add `prettier` and `npm` README badges

* dev: add pre-commit hook to format code

* refactor: move config files into a `config` directory

* dev: add styleguide and test info to CONTRIBUTING doc

* feat(array): add `partitionArray` function

* dev: automate injecting export blocks in scripts on build

* feat(array): add `flatMap` function
cahilfoley added a commit that referenced this pull request Sep 21, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants