Skip to content
This repository has been archived by the owner on Oct 10, 2018. It is now read-only.

Commit

Permalink
fix(scripts): Remove prepublish because that apparently runs on insta…
Browse files Browse the repository at this point in the history
…ll :-(

ref npm/npm#10074
  • Loading branch information
Kent C. Dodds committed Oct 26, 2015
1 parent f1e92b9 commit e24907b
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,7 @@
"deployCopy": "cp index.html deploy.ignored/ && cp dist/api-check.js deploy.ignored/dist/",
"deployClean": "rm -rf deploy.ignored/ && mkdir deploy.ignored/ && mkdir deploy.ignored/dist/",
"code-checks": "eslint src/",
"prepublish": "npm run build",
"postpublish": "publish-latest",
"semantic-release": "semantic-release pre && npm publish && semantic-release post"
"semantic-release": "semantic-release pre && npm run build && npm publish && publish-latest && semantic-release post"
},
"repository": {
"type": "git",
Expand Down

2 comments on commit e24907b

@mtscout6
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what was so confusing/frustrating about it. It only runs on install on this repo. Not when installed as a dependency to another project.

@kentcdodds
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

totally

Please sign in to comment.