npm tasks

[emmenko]

This supersedes #248

/cc @gaearon

[gaearon]

How does making a Github release fit into this workflow? Should I just do a release via web interface separately based on the pushed tag?

[emmenko]

When you do a release, a github tag is created. Then you need to manually create the release note for that existing tag (but that's always a manual process).

Or do you mean something else?

[gaearon]

Yeah this seems right.
Some tools also automate creating release notes so I just wanted to clarify.

This looks great to me.

[gaearon]

Can we just change it to %s please

[emmenko]

Sure

[emmenko]

Cool 👍

One thing though to remember: the patch/minor/major tasks work for normal semver numbers, so normal lifecycle releases.
If you want to release alpha/beta/-rc/whatever you should directly use the mversion cli (see examples), or do it manually, just to be sure.

[gaearon]

Oh, okay, thanks for explaining.

[emmenko]

@gaearon ok, I made the changes and force pushed. Should be ok now.

[gaearon]

just %s please

[emmenko]

Oh, I thought vs% without v. Will change it, sorry

[gaearon]

Why not build it as part of npm run build?

[emmenko]

The umd part? Makes sense!

[emmenko]

@gaearon feel free to do the final tweaks the way you like it ;)

[gaearon]

Sure, thanks! I'll actually use some of these for a boilerplate I'm preparing. (Tired of doing the same setup for every lib.)

[emmenko]

Tired of doing the same setup for every lib

I know! :)

PS: Glad to help.

[gaearon]

Thanks again! I'll merge tomorrow.
In the meantime, I extracted some stuff into https://github.com/gaearon/library-boilerplate

[emmenko]

Good idea! Thanks for the link.

[valscion]

Wow this PR clarifies the build process a lot and makes it easier to start hacking on redux. 👍

[mikekidder]

(Window compatibility)
npm tasks that set environment variables for Node, it requires a slight change.
For example (from/to):

"build:umd": "webpack src/index.js dist/redux.js && NODE_ENV=production webpack src/index.js dist/redux.min.js"

"build:umd": "webpack src/index.js dist/redux.js && set NODE_ENV=production&& webpack src/index.js dist/redux.min.js"

I mentioned it in gaearon/library-boilerplate#2. Should work for Mac as well (I hope) to make it all cross-platform. The no space after setting variable is the hack discovery that makes it possible in Windows.

[gaearon]

@mikekidder This doesn't pass NODE_ENV=production to Webpack on OS X for some reason.

[gaearon]

I tried to fix Windows build in #379.

[mikekidder]

Yes, "set" on OS X was questionable, I didn't have a Mac to test. I'll respond further in #379