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

etl: adjust API to use an instance method over class methods #370

Closed
jtigger opened this issue Mar 8, 2017 · 3 comments
Closed

etl: adjust API to use an instance method over class methods #370

jtigger opened this issue Mar 8, 2017 · 3 comments
Assignees

Comments

@jtigger
Copy link
Contributor

jtigger commented Mar 8, 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)

@stkent
Copy link
Contributor

stkent commented Apr 5, 2017

@jtigger ETL already appears to use instance methods rather than static methods, and has done for quite some time. Was there perhaps some other exercise you were thinking of here?

@stkent stkent self-assigned this Apr 5, 2017
@jtigger
Copy link
Contributor Author

jtigger commented Apr 7, 2017

medical marijuana? 🌱

Sorry to have created work for ya, @stkent. 😬

@jtigger jtigger closed this as completed Apr 7, 2017
@stkent
Copy link
Contributor

stkent commented Apr 7, 2017

Haha, no worries, I like instant-close issues!

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

No branches or pull requests

2 participants