Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Migrate to Jest for JS unit testing #437
Migrate to Jest for JS unit testing #437
Changes from 59 commits
652cbd7
c865bb4
1430b66
c5d6d33
820971c
44dbe10
712f1d5
183457a
8dbfc2a
d428649
601afd6
bf58866
f2f0cc1
d0eaf6e
e0729a6
803b34c
b398da2
b17c7ca
677acdd
e702b09
6ea6c73
e6e8898
e53d8c1
480011b
26623ad
b3e532f
0e872d5
8effec7
10e16b7
11dd715
5a7b14d
6a4d29d
a8f9bfc
e5d9b2d
1934d4d
3491f09
9995f71
8b3407c
c3d975f
b8f4029
c090629
37d49cd
4071ba9
a140f7b
22c4aab
02edea3
c57ffd2
53fceb3
601b91d
451b8f4
a058d2f
9abc947
5333506
cfffa24
ae11dd7
6d32ce9
c53f7e6
22382cb
cd8af25
14f7ef0
ccfeee5
78ef379
76835ca
9f24287
ab5e45b
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Didn't we extract this logic to a
react-features
helper?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 not sure that ever got merged; at any rate it was weird because even the version of React we load depends on what Gutenberg/other scripts expose. I'll have a look though. 👍
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.
This still needs to be revisited, it should import from
util/react-features
.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.
See above, this can be removed following the above change.