Shoulda-matchers: Did someone rewrite history on the rails-5 branch?

Created on 18 Sep 2017  路  7Comments  路  Source: thoughtbot/shoulda-matchers

I was pointing to edaf9cb926ee9c59c59729e7a4f8c206b44da8a1 but that commit just disappeared. 馃槩

Most helpful comment

I've gone ahead and merged rails-5 into the master branch, as I think it makes sense going forward. That should make creating new PRs easier for folks too. I won't delete the branch, but now you really can switch to master and get the same fixes.

All 7 comments

@seven1m we've ran into the same issue here - missing SHA for us is f800d62525a65f9bf001ea984ae4926a92ff5b82; 馃槩

Hi, guys!

rails-5 branch has been rebased onto master.

Pointing shoulda-matchers to master branch in Gemfile will fix it.

Ah... sorry folks, that was my bad. It slipped my mind that we've made this branch sort of public. You can keep using the rails-5 branch for your projects -- I'll refrain from doing this in the future.

So, we should be pointing to master in Rails 5 now?

@Carlows Exactly. I will update the documentation about that.

And soon version 4.0 will be released.

Well, master doesn't contain the fixes for Rails 5 -- those are all on the rails-5 branch -- so using master won't get you those fixes.

That said, I do think we should merge rails-5 into master and just remove the branch altogether. I'm generally not a fan of long-standing branches like this, and since we are focused on 4.0 there's no reason to keep two branches going. What do you think @guialbuk?

I've gone ahead and merged rails-5 into the master branch, as I think it makes sense going forward. That should make creating new PRs easier for folks too. I won't delete the branch, but now you really can switch to master and get the same fixes.

Was this page helpful?
0 / 5 - 0 ratings