fixed SequelAdapter#setup to create the table

technoweenie / guillotine

URL shortening hobby kit

http://techno-weenie.net/guillotine/

MIT License

486 stars 54 forks source link

fixed SequelAdapter#setup to create the table #5

Closed snatchev closed 12 years ago

snatchev commented 12 years ago

string had to be changed to String

technoweenie commented 12 years ago

Weird, since when? Pretty sure I had sequel tests running on this bad boy. Thanks though!

snatchev commented 12 years ago

The tests actually pass with Sqlite. It was failing under Postgres with PGError: ERROR: type "string" does not exist (Sequel::DatabaseError)

I found the fix here: http://sequel.rubyforge.org/rdoc/classes/Sequel/Database.html#method-i-create_table

Thank you for the release.

technoweenie commented 12 years ago

Oh word. Thanks for the clarification. We're not even running this on SQL, though I planned to make it super easy to deploy Guillotine to Heroku. I'd love to have it as close to a "fork and push to heroku" process as possible, actually.