Skip to content
This repository was archived by the owner on May 27, 2025. It is now read-only.

Conversation

mrbrdo
Copy link

@mrbrdo mrbrdo commented Jun 26, 2013

  • fixed migrations to be compatible with databases other than MySQL (more or less just removed backticks and used some Sequel migration methods in place where you used raw SQL)
  • fixed to use Sequel.desc and asc instead of Symbol#desc as it was deprecated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be drop_foreign_key :user_id which will also drop the column. MySQL does not automatically drop foreign keys. Also in the down action it should be add_foreign_key :user_id, :users, :key => :id according to Sequel docs:

For foreign key columns, the column in the associated table that this column references. Unnecessary if this column references the primary key of the associated table, except if you are using MySQL.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I moved away from Barkeep, but you are welcome to pick this up.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What alternative did you choose in the end? The barkeep repo seems pretty much unmaintained right now.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

naforo.com (disclaimer I am the author)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great and a git-hook based tool should work fine with our git repos which are coupled to our Redmine which handles auth/auth. I've dropped you an invite request from my company email.

OK, enough off-topic talk :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants