Skip to content

fix: adds back run.js to npm installer#1149

Merged
drager merged 1 commit intomasterfrom
avery/quick-fix
Jan 28, 2023
Merged

fix: adds back run.js to npm installer#1149
drager merged 1 commit intomasterfrom
avery/quick-fix

Conversation

@EverlastingBugstopper
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper commented Jun 13, 2022

hopefully we can get this in before the release happens otherwise i fear the npm installer may break! i meant to open this PR sooner but just never got around to it. this is the patch i submitted to the primary repo I work on for comparison to this PR.

We ended up burning two releases due to npm install woes, but things are working smoothly now. cc @drager

@EverlastingBugstopper
Copy link
Contributor Author

actually.... i'm not sure if this is needed since it looks like the release already happened and npx wasm-pack@0.10.3 works just fine for me. i'm going to close this but if something is off please reopen and ping me.

@drager
Copy link
Owner

drager commented Jun 14, 2022

I actually merged your PR after I released 0.10.3.

@EverlastingBugstopper
Copy link
Contributor Author

ah! ok! then in that case, this should likely be merged 🙈

@drager drager merged commit e3582b7 into master Jan 28, 2023
@drager drager deleted the avery/quick-fix branch March 19, 2023 09:01
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.

2 participants