Skip to content

Conversation

@oliverturner
Copy link
Collaborator

A reduced change set rebased off master

module.exports = function(env) {
var isProd = env === 'production';
var cssLoader = 'css?modules&importLoaders=1&localIdentName=ReactStyleguidist-[name]__[local]!postcss';
var cssLoader = 'css?module&importLoaders=1&localIdentName=ReactStyleguidist-[name]__[local]!postcss';
Copy link
Member

Choose a reason for hiding this comment

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

It should be modules.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

My bad... #willfix

@sapegin
Copy link
Member

sapegin commented Feb 7, 2016

That’s great, thank you very much!

The only thing I’d like to change is Lodash imports: I prefer import func from 'lodash/func' style (for tests just _ is fine).

@@ -0,0 +1 @@
v5
Copy link
Member

Choose a reason for hiding this comment

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

Is it required by Babel 6 or is it a good practice or something very useful? ;-)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not required by Babel 6, but incredibly useful to people who need to switch between versions of Node: on some projects I need to use the Node LTS version (4.2.x) to stay in sync with an environment, on others I want to switch to v5.

Using https://github.com/wbyoung/avn, simply by cd'ing into the directory I get switched to the project-specific version. I can certainly remove, but it's a nice feature for some.

@oliverturner
Copy link
Collaborator Author

Re the preference for lodash import syntax, I'd suggest that it's more in line with the direction that lodash itself is going to reference the module directly?

@sapegin
Copy link
Member

sapegin commented Feb 7, 2016

Modules, not packages. And now it’s very simple (in 3.x we had to write lodash/groupname/func, now it’s just lodash/func. I thinks it’s easier to maintain (you have to install just one lodash package) and only the required modules will be bundled by Webpack (so it’s as efficient as using multiple packages).

@oliverturner
Copy link
Collaborator Author

Removed all references to lodash.x packages and replaced _.pluck with _.map per discussion at underdash/underdash#4 (comment)

sapegin added a commit that referenced this pull request Feb 8, 2016
@sapegin sapegin merged commit 1fea5df into styleguidist:master Feb 8, 2016
@oliverturner oliverturner deleted the feature/babel6-simple branch February 21, 2016 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants