Flux utils #254

Merged
merged 9 commits into from Aug 14, 2015

Conversation

Projects
None yet
5 participants
@kyldvs
Contributor

kyldvs commented Aug 13, 2015

Release flux utils

  • update build scripts
  • add base classes
  • add an example
  • add documentation
package.json
@@ -1,6 +1,6 @@
{
"name": "flux",
- "version": "2.0.3",
+ "version": "2.1.0",

This comment has been minimized.

@zpao

zpao Aug 14, 2015

Member

Do this in a separate commit, not a part of this PR (better for tagging releases)

@zpao

zpao Aug 14, 2015

Member

Do this in a separate commit, not a part of this PR (better for tagging releases)

This comment has been minimized.

@kyldvs

kyldvs Aug 14, 2015

Contributor

will do 👍

@kyldvs

kyldvs Aug 14, 2015

Contributor

will do 👍

fisherwebdev added a commit that referenced this pull request Aug 14, 2015

Merge pull request #254 from kyldvs/flux-utils
Flux utils

Amazing work, Kyle!

Users of Flux.js should note that index.js does not include the new utility files, so they are not included in a normal build.

@fisherwebdev fisherwebdev merged commit 5260d75 into facebook:master Aug 14, 2015

@kyldvs kyldvs deleted the kyldvs:flux-utils branch Aug 17, 2015

@oveddan

This comment has been minimized.

Show comment
Hide comment
@oveddan

oveddan Sep 29, 2015

@kyldvs can you explain this syntax, with type and the generic arguments for OrderedMap? I can't seem to find documentation on that anywhere.

@kyldvs can you explain this syntax, with type and the generic arguments for OrderedMap? I can't seem to find documentation on that anywhere.

This comment has been minimized.

Show comment
Hide comment
@kyldvs

kyldvs Sep 29, 2015

Contributor

Sure, these are just type aliases from facebook/flow, see: http://flowtype.org/docs/type-aliases.html

Contributor

kyldvs replied Sep 29, 2015

Sure, these are just type aliases from facebook/flow, see: http://flowtype.org/docs/type-aliases.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment