[openstreetmap/openstreetmap-website] Added workflow for PR labeling using Danger (PR #4988)

Nenad Vujicic notifications at github.com
Fri Jul 19 13:39:24 UTC 2024


Dear @tomhughes ,

Thanks for reviewing and comments. I've just updated with above fixes. However, I'm still getting strange error during checking about setting up database. Can you please restart these checks?

Regarding linear_history plugin, its implementation is exactly the same we are using in Dangerfile and it reports "Please rebase to get rid of the merge commits in this PR", so, is it better to install new gem for perhaps simpler code or to use current solution? I would be happy to replace with using linear_history plugin if required :-)

Please, let me know if I could improve these further.

Thanks,
Nenad.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4988#issuecomment-2239206473
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4988/c2239206473 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240719/a79e928a/attachment.htm>


More information about the rails-dev mailing list