publish to npm script is broken

google / traceur-compiler

Traceur is a JavaScript.next-to-JavaScript-of-today compiler

Apache License 2.0

8.17k stars 580 forks source link

publish to npm script is broken #2070

Closed arv closed 8 years ago

arv commented 8 years ago

I tried to run npm run just-publish and I ran into a lot of trouble.

"pre-" needs to be "pre" in package.json
build/printSemver.js no longer exists.

I tried a couple of things and the npm version is increased...

I should have realized that I could have used npm publish --dry-run earlier to prevent all these spurious version updates.

johnjbarton commented 8 years ago

Sorry about that, our publish is pretty dicey. I'm not sure my improvements were... At least in part this is a side-effect of my fix for #2054.

johnjbarton commented 8 years ago

I gather that you did publish to npm to push the version number up to 0.0.100. Unfortunately now npm install traceur fails with Refusing to install traceur as a dependency of itself. I'll try to sync it back up.

arv commented 8 years ago

Sorry, I had to bump the version to get the script through... but I should have used --dry-run from the start. Lesson learned.

johnjbarton commented 8 years ago

I am not able to make clean because npm install fails.

We depend upon chai, chai has a dev dep on karma, karma fails

208 error code EPEERINVALID
209 error peerinvalid The package karma@0.12.37 does not satisfy its siblings' peerDependencies requirements!
209 error peerinvalid Peer karma-phantomjs-launcher@0.1.1 wants karma@>=0.9
210 verbose exit [ 1, true ]

johnjbarton commented 8 years ago

npm run just-publish seems to work.