Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

Add a creatable parameter to enable Creatable select components to be…#94

Merged
wyuenho merged 5 commits intomasterfrom
enable-creatable-select
Mar 16, 2017
Merged

Add a creatable parameter to enable Creatable select components to be…#94
wyuenho merged 5 commits intomasterfrom
enable-creatable-select

Conversation

@marksteyn
Copy link
Contributor

… created

"dependencies": {
"react": "^0.14.2 || ^15.0.0-0",
"react-select": "1.0.0-rc.2 || 1.x"
"react-select": "1.0.0-rc.3 || 1.x"
Copy link
Contributor

Choose a reason for hiding this comment

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

I realize it was RC 2 before but shouldn't we be depending on a more stable version? Does the functionality we need exist in 1.0.0+?

@tajo
Copy link
Contributor

tajo commented Mar 14, 2017

Please rebase.

@marksteyn marksteyn force-pushed the enable-creatable-select branch from c661ef5 to 795cd09 Compare March 15, 2017 13:30
@wyuenho
Copy link
Contributor

wyuenho commented Mar 15, 2017

2 things concerning here. I've tested this on the latest nvm, node 6.10.0, and npm 3 and 4.

  1. ["transform-es2015-modules-commonjs", { "loose": true }] should make transpiling the src files fail during test runs, somehow CI doesn't show this.
  2. CI shows the same test result as if loose: false, but a number of us can't reproduce this on our local machines.

@wyuenho wyuenho merged commit b0050af into master Mar 16, 2017
@wyuenho wyuenho deleted the enable-creatable-select branch March 17, 2017 23:04
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.

5 participants