search

ga-wdi-boston / ruby-enumerable-io

Other
0 stars 126 forks source link

Change starter code for pets_array.rb #17

Open raq929 opened 7 years ago

raq929 commented 7 years ago 
- pets = Pets.new('data/pets.csv').pets
+ pets = Pets.new.pets
gaand commented 7 years ago

Why? The point here is that the source of the data might be different.

jrhorn424 commented 7 years ago

The current solution code needs to be updated then. It no longer takes an argument, to parallel People.

gaand commented 7 years ago

Ah. I haven't thought about this in a while.

If we're going with Pet.all loading and returning everything, then the file shouldn't be passed in.

Maybe a config/manifest for where to find things?

jrhorn424 commented 7 years ago

@gaand How do you feel about YAML?

gaand commented 7 years ago

I'm fine with whatever mechanism we use as long as it doesn't become a learning objective.