Skip to content

accumulate: adjust API to use an instance method over class methods #361

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

Closed
jtigger opened this issue Mar 7, 2017 · 2 comments
Closed

Comments

@jtigger
Copy link
Contributor

jtigger commented Mar 7, 2017

As per #177, adjust the test suite to require a solution that uses an instance method instead of a class method.

(see #344 as an example and the conversation about Function Object vs. Value Object)

@jtigger jtigger added the code label Mar 7, 2017
@stkent
Copy link
Contributor

stkent commented Mar 7, 2017

Reminder, accumulate is deprecated in the process of being deprecated :)

@jtigger
Copy link
Contributor Author

jtigger commented Mar 8, 2017

oh yeah, totally forgot... just blindly combing through the track. Thank you!

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

No branches or pull requests

2 participants