[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