Skip to content

New sample types - #445

Merged
zerobias merged 4 commits into
effector:masterfrom
doasync:new-sample-types
Jan 27, 2021
Merged

New sample types#445
zerobias merged 4 commits into
effector:masterfrom
doasync:new-sample-types

Conversation

@doasync

@doasync doasync commented Jan 24, 2021

Copy link
Copy Markdown
Member

No description provided.

@netlify

netlify Bot commented Jan 24, 2021

Copy link
Copy Markdown

Deploy request for effector-docs pending review.

Review with commit b644373

https://app.netlify.com/sites/effector-docs/deploys

expect(typecheck).toMatchInlineSnapshot(`
"
no errors
Expected 2 type arguments, but got 1.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

regression

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I believe I can solve it)

@doasync doasync changed the title New sample types (for generated type tests) New sample types (proof of concept) Jan 24, 2021
@doasync doasync changed the title New sample types (proof of concept) New sample types Jan 26, 2021
@zerobias
zerobias marked this pull request as ready for review January 27, 2021 01:39
@zerobias

Copy link
Copy Markdown
Member

Fantastic work! This going to be a huge improvement in developer experience 🚀

@zerobias
zerobias merged commit 1f1fa76 into effector:master Jan 27, 2021
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.

2 participants