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

[WIP] reroute npm publish to deploy scripts #190

Closed
wants to merge 2 commits into from

Conversation

jbaxleyiii
Copy link
Contributor

@jbaxleyiii jbaxleyiii commented May 8, 2016

Now npm publish is the way to publish instead of npm run deploy

Locally running npm i will print out npm run deploy. This won't actually deploy but is required in order to route npm publish to our custom script.

On the npm shipped version, we remove all of the scripts so npm run deploy won't happen on anyone else's machine.

Fixes #188
.

@stubailo
Copy link
Contributor

We should fix the build on Windows.

@jbaxleyiii jbaxleyiii self-assigned this May 15, 2016
@jbaxleyiii jbaxleyiii changed the title reroute npm publish to deploy scripts [WIP] reroute npm publish to deploy scripts May 17, 2016
@helfer
Copy link
Contributor

helfer commented Jun 2, 2016

What's the status on this PR? Is there anything I (or someone else) can help with?

@stubailo
Copy link
Contributor

stubailo commented Jun 29, 2016

Closing in favor of removing the special deploy script as part of #324

@stubailo stubailo closed this Jun 29, 2016
@stubailo stubailo deleted the error-on-publish branch September 20, 2016 03:43
jbaxleyiii pushed a commit that referenced this pull request Oct 18, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants