[openstreetmap/openstreetmap-website] Improvements to danger workflow (PR #5290)

Tom Hughes notifications at github.com
Sat Oct 26 11:28:23 UTC 2024


This gets the danger workflow working by switching to my fixed version (current open upstream as https://github.com/danger/danger/pull/1501) and also tweaks a few other things:

* Renames the workflow to reflect that this is not just about labelling
* Drops the `run-name` so that he runs will be names after the PR instead of all having the same name
* Adjusts the label names to be consistent with our other labels.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5290

-- Commit Summary --

  * Get danger working in GitHub actions
  * Avoid suggesting danger is only about labels
  * Make danger labels use consisent capitalisation

-- File Changes --

    R .github/workflows/danger.yml (4)
    M Dangerfile (12)
    M Gemfile (2)
    M Gemfile.lock (36)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5290.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5290.diff

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

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


More information about the rails-dev mailing list