Skip to content

(fix) - passing null to forwardRef#2600

Merged
marvinhagemeister merged 4 commits intomasterfrom
fix-null-ref
Jul 6, 2020
Merged

(fix) - passing null to forwardRef#2600
marvinhagemeister merged 4 commits intomasterfrom
fix-null-ref

Conversation

@JoviDeCroock
Copy link
Copy Markdown
Member

fixes: #2599

Comment thread compat/src/forwardRef.js
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 1, 2020

Size Change: +6 B (0%)

Total Size: 40.1 kB

Filename Size Change
compat/dist/compat.js 3.2 kB +3 B (0%)
compat/dist/compat.module.js 3.23 kB +1 B
compat/dist/compat.umd.js 3.25 kB +2 B (0%)
ℹ️ View Unchanged
Filename Size Change
debug/dist/debug.js 2.99 kB 0 B
debug/dist/debug.module.js 2.98 kB 0 B
debug/dist/debug.umd.js 3.08 kB 0 B
devtools/dist/devtools.js 184 B 0 B
devtools/dist/devtools.module.js 195 B 0 B
devtools/dist/devtools.umd.js 260 B 0 B
dist/preact.js 3.96 kB 0 B
dist/preact.min.js 3.98 kB 0 B
dist/preact.module.js 3.97 kB 0 B
dist/preact.umd.js 4.01 kB 0 B
hooks/dist/hooks.js 1.09 kB 0 B
hooks/dist/hooks.module.js 1.11 kB 0 B
hooks/dist/hooks.umd.js 1.17 kB 0 B
test-utils/dist/testUtils.js 437 B 0 B
test-utils/dist/testUtils.module.js 439 B 0 B
test-utils/dist/testUtils.umd.js 515 B 0 B

compressed-size-action

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 6, 2020

Coverage Status

Coverage remained the same at 99.798% when pulling 50cc1ca on fix-null-ref into 6b503aa on master.

Copy link
Copy Markdown
Member

@marvinhagemeister marvinhagemeister left a comment

Choose a reason for hiding this comment

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

Love this! 🙌 The test via a direct function call to copy what mobx is doing internally is very elegant 🥇

@marvinhagemeister marvinhagemeister merged commit ff0bf29 into master Jul 6, 2020
@marvinhagemeister marvinhagemeister deleted the fix-null-ref branch July 6, 2020 15:59
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.

got error after updating preact version from 10.4.4 to 10.4.5

3 participants