[openstreetmap/openstreetmap-website] Add OpenMapTiles vector layer (PR #3753)

[Tom Hughes]

You can't really squash merge something that itself has merge commits, or at least I wouldn't like to.

We don't generally recommend merging into a PR branch - if you want a newer version of master just rebase instead as it makes it much easier to review.

Likewise there is really no point in committing a revert of a revision in a PR branch when you can just use rebase to delete the erroneous commit which will again minimise the effort needed to review things.

More generally if you realise an earlier commit isn't quite then just use rebase to fix it so that reviewers can see a clean series of changes that make logical sense rather than having to jump back and forth between commits to understand things.

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

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