Skip to content
This repository was archived by the owner on Jul 19, 2019. It is now read-only.

Pass default boolean value to attribute #27

Closed
wants to merge 1 commit into from
Closed

Pass default boolean value to attribute #27

wants to merge 1 commit into from

Conversation

artyomtrityak
Copy link
Contributor

Due to the react/jsx-boolean-value eslint configuration we need to pass default boolean value. This is common pattern and it will be better to make sure tutorial teaches users good code style.

@ryanflorence
Copy link
Contributor

We changed the repo structure to make way more sense, previously the git history was part of the "structure" and that was a terrible idea, making PRs impossible to merge.

Now each lesson has its own directory so we can just click the merge button!

Mind updating your PR to work with the current code on the master branch? Thanks!

@artyomtrityak
Copy link
Contributor Author

closing this due to my new pull request #118

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.

2 participants