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

Changed if ! to unless #9953

wants to merge 1 commit into


None yet
2 participants

Changed if ! to unless for more readability


rafaelfranca commented Mar 27, 2013

This fall in the cosmetic changes and we prefer to not accept they.

Changes that are cosmetic in nature and do not add anything substantial to the stability, functionality, or testability of Rails will generally not be accepted.


Thank you for the pull request.

@prathamesh-sonpatki prathamesh-sonpatki deleted the prathamesh-sonpatki:AR-unless branch Mar 27, 2013

@rafaelfranca Ah. I missed this before this PR.
cool 😄

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