Skip to content
This repository was archived by the owner on Oct 23, 2023. It is now read-only.

Conversation

@TimKolberger
Copy link
Collaborator

@TimKolberger TimKolberger commented Nov 14, 2022

I ported the polymorphic factory to solid and used the built in Dynamic component.
Seems to work as good as with React 🎉

Event types in callbacks e.g. onClick still have the old component type

@TimKolberger TimKolberger marked this pull request as ready for review November 14, 2022 20:31
@TimKolberger TimKolberger changed the base branch from main to feat/explicit-types-and-exports November 14, 2022 20:31
@codecov
Copy link

codecov bot commented Nov 14, 2022

Codecov Report

Merging #27 (ddc01ca) into main (dd062b4) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main       #27    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files            3         5     +2     
  Lines          145       269   +124     
  Branches        10        18     +8     
==========================================
+ Hits           145       269   +124     
Impacted Files Coverage Δ
packages/react/src/forwardRef.tsx 100.00% <100.00%> (ø)
packages/react/src/index.ts 100.00% <100.00%> (ø)
packages/react/src/polymorphic-factory.tsx 100.00% <100.00%> (ø)
packages/solid/src/index.ts 100.00% <100.00%> (ø)
packages/solid/src/polymorphic-factory.tsx 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Base automatically changed from feat/explicit-types-and-exports to main November 15, 2022 10:30
@TimKolberger TimKolberger merged commit eb3d8a1 into main Nov 15, 2022
@TimKolberger TimKolberger deleted the feat/solid branch November 15, 2022 11:43
@github-actions github-actions bot mentioned this pull request Nov 15, 2022
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.

1 participant