<p>Changing vendored assets independently of upstream is unacceptable in any case so we certainly won't be able to take that part. Those will (if accepted upstream) filter through in due course.</p>
<p>That is another advantage of having the pipeline do it of course - it is acceptable for that to publish an optimised version of a third party image.</p>
<p>I believe image_optim was only really added to optimise the svg images so we just disabled everything else rather than having to try and validate all the other ones and make sure they didn't break anything. Enabling them will also require making sure that the necessary tools are installed by chef on the production servers but that is easy enough so long as they are packaged for ubuntu.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1691#issuecomment-347713596">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLUsq7kgAK_gXP_-PnNrWrVJnMbYVks5s7KhugaJpZM4QuAHd">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLZhoWvDfTuXcb9QyxXSZ5Kv4Y99fks5s7KhugaJpZM4QuAHd.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/pull/1691#issuecomment-347713596"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@tomhughes in #1691: Changing vendored assets independently of upstream is unacceptable in any case so we certainly won't be able to take that part. Those will (if accepted upstream) filter through in due course.\r\n\r\nThat is another advantage of having the pipeline do it of course - it is acceptable for that to publish an optimised version of a third party image.\r\n\r\nI believe image_optim was only really added to optimise the svg images so we just disabled everything else rather than having to try and validate all the other ones and make sure they didn't break anything. Enabling them will also require making sure that the necessary tools are installed by chef on the production servers but that is easy enough so long as they are packaged for ubuntu."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1691#issuecomment-347713596"}}}</script>