Pin mapbox to 0.52

[Rich-Harris]

A new Mapbox GL version breaks the context API tutorial — will need to pin to 0.52, or figure out how to serve 0.53 from bundle.run

• https://bundle.run/[email protected]
• https://bundle.run/[email protected] (errors)

[Conduitry]

Bonus if we do manage to get it to work with 0.53.1: You can now have mapbox tokens that can only be used at specific domains or URLs.

[mourner]

https://bundle.run/[email protected] works. BTW, any reason to use bundle.run instead of the official build at https://api.tiles.mapbox.com/mapbox-gl-js/v0.53.1/mapbox-gl.js?

[Rich-Harris]

I'm seeing this error, are you getting something else?

bundle.run is just what the REPL uses when you do import whatever from 'whatever'. We could certainly do import mapbox from 'https://api.tiles.mapbox.com/mapbox-gl-js/v0.53.1/mapbox-gl.js' but I think it's better if we can show the code that a user would write in their own app, which would generally import from node_modules.

[Conduitry]

I'm getting that same error on 0.53.1 on bundle.run. On the demo instance https://packd.now.sh/[email protected] however - after a lot of churning - I did get a bundle back. I don't know what that means.

[mourner]

@Rich-Harris yep, getting something else!

So maybe this is an error that happened for some CDN regions of bundle.run but not others, not sure... Certainly doesn't seem to be an issue with GL JS specifically, just some erratic bundler crash.

[Rich-Harris]

My favourite kind of bug 🙃