Skip to content

Comments

Add more test coverage for custom elements#11811

Merged
gaearon merged 1 commit intofacebook:masterfrom
gaearon:custom-el-coverage
Dec 8, 2017
Merged

Add more test coverage for custom elements#11811
gaearon merged 1 commit intofacebook:masterfrom
gaearon:custom-el-coverage

Conversation

@gaearon
Copy link
Collaborator

@gaearon gaearon commented Dec 8, 2017

Nothing super important, just noticed we don't cover some of the existing behavior which might be a bit odd, but it existed since 16.0. So we shouldn't break it refactoring that code.

@gaearon
Copy link
Collaborator Author

gaearon commented Dec 8, 2017

(In particular, the fact that custom attributes aren't subject to any restrictions in our logic whatsoever right now.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants