hlfan left a comment (openstreetmap/openstreetmap-website#6726)
> I think it's probably a bit silly moving `CombinedControlGroup` to
> `maplibre/dom_util` in one commit and then moving it again to
> `maplibre/controls` in the next one - why not just move it directly to the
> final location in the first commit?
I wanted to get reviews in early; I also felt it was silly when committing this
but figured I'd clean it up together if other restructures were suggested.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6726#issuecomment-3770123418
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6726/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
