search

blambeau / rqp2

The Relational Query Puzzle Platform
2 stars 1 forks source link

Seed does not take current year as default value #2

Closed ancailliau closed 8 years ago

ancailliau commented 8 years ago

Typing bin/rake db:seed does not seed the database with the folder corresponding to the current year.

blambeau commented 8 years ago

I agree, it should. See here: https://github.com/blambeau/rqp2/blob/master/tasks/db.rake#L25-L34

Still, it would be easy to replace '2014' by Time.now.year.to_s but that does not follow the academic year (I suppose the current seeds are in 2015, not 2016, right?)

ancailliau commented 8 years ago

Nope, I've put them in 2016. Fixed in the proposed commit.

blambeau commented 8 years ago

@ancailliau OK, great, please submit a pull request then, I'll merge that.

P.S. Possibly we want to add the 2016 seeds to the source code too?