[openstreetmap/openstreetmap-website] WIP: devcontainer configuration (PR #6424)

Marwin Hochfelsner notifications at github.com
Wed Oct 8 12:17:45 UTC 2025


@hlfan commented on this pull request.



> @@ -0,0 +1,17 @@
+name: devcontainer
+
+on:
+  - push
+  - pull_request
+

So I've now tried matching the style of `docker.yml`, `lint.yml`, and `tests.yml`, and, locally, Git does NOT conflate the history of the workflows then either. So if you've been keeping these around for that reason, it's unnecessary. And as Tom already suggested, rebasing the commits as "one to add devcontainer and the one to remove docker" would eliminate the chance of this happening accidentally anyway.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6424/review/3314547012 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251008/28b0f64f/attachment.htm>


More information about the rails-dev mailing list