<p></p>
<p dir="auto">Thanks for splitting this into a separate PR, since changes to yarn are unrelated to change to bundler.</p>
<p dir="auto">At first glance I have some serious concerns about this PR, so it would be good to update this PR with a lot more information. For example, what is the purpose of the upgrade? Does it fix anything in particular, is the old version of yarn blocking you from something, or is it just a maintenance task, or something else? (All are valid answers!)</p>
<p dir="auto">Additionally:</p>
<ul dir="auto">
<li>What is the 2.0MB of javascript in .yarn/releases ? Is this a full copy of yarn? If so, why do we need to have it in our own repository? We try to avoid copy+pasting external dependencies into this repo, and aim to only have code here that we intend to maintain ourselves. Why not use distribution packages for yarn like we do now?</li>
<li>What implications are there for moving away from node_modules? Are all of our modules going to work under the alternative yarn system? Will it impact us when we want to add more modules in the future?</li>
<li>Why is the --ignore-engines flag removed? It was added for a reason (see <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="954646181" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/3275" data-hovercard-type="issue" data-hovercard-url="/openstreetmap/openstreetmap-website/issues/3275/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/issues/3275">#3275</a> ) and so removing it should explain why it's no longer necessary.</li>
</ul>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3515#issuecomment-1080828281">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPRGMEEFQNITXTV6MTVCHJIHANCNFSM5R3NX3OA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLLXIXUWWDYMVZT4D63VCHJIHA5CNFSM5R3NX3OKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIBWCC6I.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/3515/c1080828281</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/3515#issuecomment-1080828281",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3515#issuecomment-1080828281",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>