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

Npm publish #215

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Conversation

CedricGuillemet
Copy link

New manual GitHub actions workflow that builds and publish an NPM package.
My indend is then to use unpkg tp expose tinyusdz and do usdz loading in Babylon.js BabylonJS/Babylon.js#16005

tinyusdz NPM should be own (and created) by its maintainer (IMHO).

@syoyo
Copy link
Collaborator

syoyo commented Dec 17, 2024

Awesome!

We have no experience of publishing a package to NPM yet, so need to learn how NPM packaging works firstly.

Let us give some time to learn NPM packaging and review the PR.

@syoyo syoyo added the enhancement New feature or request label Dec 17, 2024
@CedricGuillemet
Copy link
Author

you can test locally the npm package if you download the artifact. and you can test the publishing dry run directly from the github action parameters. It's very useful!

@syoyo
Copy link
Collaborator

syoyo commented Dec 18, 2024

you can test the publishing dry run directly from the github action parameters. It's very useful!

😍

@syoyo
Copy link
Collaborator

syoyo commented Dec 20, 2024

I have created NPM package tinyusdz

https://www.npmjs.com/package/tinyusdz

And also added NPM secret token to Github Actions.

This PR will be soon merged(after some workflow file review)

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

Successfully merging this pull request may close these issues.

2 participants