On Tuesday, March 15, 2016 at 4:14:43 PM UTC+1, Jeremy Evans wrote: > > On Tuesday, March 15, 2016 at 12:12:43 AM UTC-7, Ilja Resch wrote: >> >> Glad to contribute. But there is still the problem with *text* and >> *next. * >> ALTER TABLE [test_migration] ALTER COLUMN [text_column] text(2147483647) >> NULL >> ALTER TABLE [test_migration] ALTER COLUMN [ntext_column] ntext(1073741823 >> ) NULL >> >> The column size is wrong. >> *Text* and *ntext* columns should not have any size behind the column >> type. Possible fix would be the addition of: >> !["text", "ntext"].include?(type) >> in >> if [:string, :decimal].include?(sch[:type]) and !["text", "ntext"]. >> include?(type) and size = (sch[:max_chars] || sch[:column_size]) >> >> Thanks >> > > Sorry, I missed that. That's why it's best if you send a diff instead of > just new code. :) I'll commit a fix in a little bit. Thanks for the help! > > Thanks, > Jeremy >
My fault, next time i'll do Thanks -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/sequel-talk. For more options, visit https://groups.google.com/d/optout.
