Skip to content

Remove unused parameter #9

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 20, 2016
Merged

Remove unused parameter #9

merged 1 commit into from
Dec 20, 2016

Conversation

@jamescostian
Copy link
Contributor

@jamescostian jamescostian commented Dec 20, 2016

The function flattenFrom calls flattenDown with 3 parameters, however, the function flattenDown only accepts and uses 2. The final parameter (Infinity) isn't needed.

@jamescostian
Remove unused parameter
d6b822b 
The function flattenFrom calls flattenDown with 3 parameters, however, the function flattenDown only accepts and uses 2. The final parameter (Infinity) isn't needed
@coveralls
Copy link

coveralls commented Dec 20, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling d6b822b on jamescostian:remove_unnecessary_parameter into fc56f6b on blakeembrey:master.

@blakeembrey blakeembrey merged commit 9d75ea0 into blakeembrey:master Dec 20, 2016
@blakeembrey
Copy link
Owner

blakeembrey commented Dec 20, 2016

Thanks 😄

@jamescostian jamescostian deleted the remove_unnecessary_parameter branch December 20, 2016 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jamescostian @coveralls @blakeembrey