Any reason you chose to use `::` rather than `.` to send the `warn_about_should!` message?  I prefer to use `.` for message sends everywhere even though ruby allows `::` for this case.