Chalk

[astridx]

Launch Checklist

See #715

Output of npm run test-expressions

Before

After

I wonder why this change does not show up in package.lock. I have called npm ci. Is that OK for a removal?

• [ x ] Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!

[github-actions]

Bundle size report:

Size Change: 0 B
Total Size Before: 194 kB
Total Size After: 194 kB

Output file	Before	After	Change
maplibre-gl.js	185 kB	185 kB	0 B
maplibre-gl.css	9.49 kB	9.49 kB	0 B

ℹ️ View Details

No major changes

[wipfli]

Use npm uninstall chalk.

[birkskyum]

Given that it's already removed from package.json, you will have to rebase to main and run npm install with node 16.

[astridx]

Thank you @wipfli and @birkskyum .
Now this should be fine.

[HarelM]

What's the motivation on removing this package? It seems useful, isn't it?

[astridx]

What's the motivation on removing this package? It seems useful, isn't it?

Less dependencies . See #699 (comment)

[HarelM]

Can you try using console.warn and console.error instead if yellow and red?

[astridx]

npm run test-expressions

I tried it, but it does not change the color in my console. See https://stackoverflow.com/questions/51449019/what-is-the-difference-between-console-log-and-console-error-in-js/51449060#51449060