Skip to content

Add argument to rolify method to pass block through to HABTM roles to…#496

Open
OtherCroissant wants to merge 2 commits intoRolifyCommunity:masterfrom
OtherCroissant:master
Open

Add argument to rolify method to pass block through to HABTM roles to…#496
OtherCroissant wants to merge 2 commits intoRolifyCommunity:masterfrom
OtherCroissant:master

Conversation

@OtherCroissant
Copy link
Copy Markdown

@OtherCroissant OtherCroissant commented Oct 1, 2018

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 1, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 92.516% when pulling 4a185e7 on CUnknown:master into bb3d445 on RolifyCommunity:master.
@coveralls
Copy link
Copy Markdown

coveralls commented Oct 1, 2018

Coverage Status

Coverage increased (+1.0%) to 93.284% when pulling db0ebec on CUnknown:master into 522c0d7 on RolifyCommunity:master.
EppO
EppO requested changes Feb 27, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@EppO EppO left a comment

Choose a reason for hiding this comment

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

Would need to add a test for that use case
@EppO EppO added the tests label Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests

3 participants

@OtherCroissant @coveralls @EppO