-
Notifications
You must be signed in to change notification settings - Fork 65
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
Add browserify outputs to package #26
Comments
I don't think this should block the release. We'll need to do a 1.0.1 once we figure it out, but releases are cheap. |
ok, I've confirmed that everything should work if we add |
Before:
after:
I think it's worth the extra size for the CDN convenience. |
This worked fine, but now we need to change the order of operations for release. You'll notice that https://unpkg.com/peggy@1.0.0-rc.1/browser/peggy.min.js still says |
OK, I just did rc2 using the release script. I like how simple it is. I suggest you don't use the script unless you have 2FA enabled on npm, because that gives you one last chance to bail before you can't recover. |
use the browser field in package.json? WIll unpkg.com use that?
The text was updated successfully, but these errors were encountered: