Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run build-examples #21588

Merged
merged 1 commit into from
Apr 6, 2021
Merged

Run build-examples #21588

merged 1 commit into from
Apr 6, 2021

Conversation

marcofugaro
Copy link
Contributor

Related issue: #21584

Description

Run the build-examples script introduced in #21584.

@Mugen87
Copy link
Collaborator

Mugen87 commented Apr 6, 2021

FYI: The additional lgtm errors are produced since RGBMLoader (to be more precise the embedded UPNG.js) is now available in examples/js, too.

@mrdoob mrdoob added this to the r128 milestone Apr 6, 2021
@mrdoob mrdoob merged commit d9a5084 into mrdoob:dev Apr 6, 2021
@mrdoob
Copy link
Owner

mrdoob commented Apr 6, 2021

Thanks!

@fraguada
Copy link
Contributor

@marcofugaro @Mugen87 @mrdoob this PR added a /js version of the 3dmLoader. Was this the intention? 3dmLoader was developed as a jsm-first/only loader 😎.

@mrdoob
Copy link
Owner

mrdoob commented Apr 15, 2021

@fraguada jsm/ files are now the canonical ones and we are able to generate js/ from them 👍

@fraguada
Copy link
Contributor

That's rad. I was refactoring the jsm version and was wondering about the js version. I'll leave it as it then and let the build tools work their magic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants