[openstreetmap/openstreetmap-website] Added a validation to the Redaction model for the title. (#2034)

Andy Allan notifications at github.com
Wed Oct 24 06:54:52 UTC 2018


> If you're going to open a PR there's no need to open a bug first only to then open a PR that fixes it - that just causes the information to be fragmented over two tickets when one would do.
> 
> And if you do open a bug first then please at least mark the PR as closing the bug in your commit message so that we don't have to close it manually...

I don't think these chastisements are necessary, nor particularly welcoming for a first time contributor.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2034#issuecomment-432535270
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181023/cda2056b/attachment.html>


More information about the rails-dev mailing list