-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid in-place mutation of the attributes of Router's children #169
Conversation
All credit for this change goes to @developit (who sent it over Slack)
@@ -0,0 +1,12 @@ | |||
import { cloneElement } from 'preact'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put this file in a new test_helpers
directory, because it's not a test itself, but it's imported by some of the tests. If you'd prefer it live within the test
directory, or a subdirectory inside test
(e.g. test/helpers
), I'd be happy to move it there. We'll just need to modify files: ['test/**/*.js']
in the karma config to exclude this if we move it inside test
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with how you laid it out :)
@@ -0,0 +1,12 @@ | |||
import { cloneElement } from 'preact'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with how you laid it out :)
Fixes #163