-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support :if and :unless for track_history callbacks #212
Support :if and :unless for track_history callbacks #212
Conversation
2 similar comments
@dblock I'd appreciate if you can take a moment to merge 212, 211, and 210 and ideally cut a release. Both Jagdeep and myself have reviewed these. |
Done. And this looks great, rebase/merge along. |
@jagdeepsingh please handle |
This will be conflicted again after merging #211. Will rebase it then. |
647f606
to
dcdf689
Compare
Rebased to latest master. @dblock Please merge. |
Great, merged. |
@dblock thanks for all your help. Can I ask you to cut an official release? |
I can try to get to it in the next day or so, otherwise you can cut one too :) |
Raised for #209 with RSpecs here.