You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed Source/Scene/Globe.js is importing Material.js but isn't using it in the code aside from referring to it in the doc comments. This is causing lots of code to be included in my builds unnecessarily...
Would a pull request removing this import be welcome?
The text was updated successfully, but these errors were encountered:
Thanks @puckey, yes it looks like that can be removed. Feel free to open a PR removing that and any other unnecessary requires. We usually go through maybe once a year and clean that up, but it's been a little while
I noticed Source/Scene/Globe.js is importing Material.js but isn't using it in the code aside from referring to it in the doc comments. This is causing lots of code to be included in my builds unnecessarily...
Would a pull request removing this import be welcome?
The text was updated successfully, but these errors were encountered: