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

Emin Kocan notifications at github.com
Tue Aug 13 01:47:31 UTC 2024


@kcne commented on this pull request.

It would be beneficial to incorporate a check for our [commit conventions](https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md#committing) in this PR. Ensuring adherence to these conventions can help maintain consistency across our codebase.

I would also suggest adding pre-commit hooks to the repository. This addition could ease the maintenance workload and guide contributions and commits in the right direction before they are even made. Although I'm not fully versed in how pre-commit hooks integrate with forks, I'm willing to explore this further if you would be open to merging such a change.



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

Message ID: <openstreetmap/openstreetmap-website/pull/4988/review/2234310291 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240812/1887bad0/attachment.htm>


More information about the rails-dev mailing list