Skip to content

Conversation

@reinink
Copy link
Member

@reinink reinink commented Nov 3, 2017

Removes the rest operator (...) from the documentation's Tailwind config, making it possible to build the docs on older versions of Node.

@reinink reinink requested a review from adamwathan November 3, 2017 16:22
config.colors = {
...config.colors,

config.colors = Object.assign(config.colors, {
Copy link
Member

Choose a reason for hiding this comment

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

Object.assign mutates the first object in the parameter list, so it isn't technically required to assign this back to the property. I can't decide if I think it's harmless or misleading.

If you want to do this in an immutable way, you can rewrite it like:

config.colors = Object.assign({}, config.colors, { ... })

Note the extra empty anonymous object as the first param.

Copy link
Member

Choose a reason for hiding this comment

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

Also this is just the docs config file so who fucking cares.

@adamwathan adamwathan merged commit 4f7e99c into master Nov 3, 2017
@adamwathan adamwathan deleted the make-docs-config-compatible-with-older-versions-of-node branch November 3, 2017 18:24
DCzajkowski pushed a commit to DCzajkowski/tailwindcss that referenced this pull request Jul 23, 2019
…grow-0

Rename the `flex-no-[grow/shrink]` classnames to `flex-[grow/shrink]-0` (tailwindlabs#687)
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.

4 participants