GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Fixes issue #138.
I don' t think you need to strip here.
If you don't, it will automatically add a blank space when there is not a role descruption.
what about [base_description, role_description].compact.join(' ') (unless one of them could be an empty string...)?
[base_description, role_description].compact.join(' ')
the compact.join(' ') does work, any opinions on why (or if) its better than join(' ').strip?
It's not better, just an alternative way of doing things. If it doesn't work though, I've no problem joining and striping.
Updated description of AllowMassAssignmentMatcher.
👍, merge it.