Skip to content
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

Updated "Using typed injections" for better type safety #463

Merged
merged 4 commits into from
Sep 24, 2022

Conversation

jankalfus
Copy link
Contributor

@jankalfus jankalfus commented Mar 29, 2020

Updated the "Using typed injections" section of the typescript tutorial to provide a more tightly bound example of typing the store injections. I believe this is more type safe than the original in the sense that there's now a single source of truth.

@codecov
Copy link

codecov bot commented Mar 29, 2020

Codecov Report

Merging #463 (a6e67ee) into master (5c4c0e8) will increase coverage by 2.75%.
The diff coverage is n/a.

❗ Current head a6e67ee differs from pull request most recent head d50824d. Consider uploading reports for the commit d50824d to get more accurate results

@@            Coverage Diff             @@
##           master     #463      +/-   ##
==========================================
+ Coverage   94.97%   97.72%   +2.75%     
==========================================
  Files          18       20       +2     
  Lines         696      571     -125     
  Branches      139      104      -35     
==========================================
- Hits          661      558     -103     
+ Misses         30       11      -19     
+ Partials        5        2       -3     
Impacted Files Coverage Δ
src/computed-properties.js 80.95% <0.00%> (-19.05%) ⬇️
src/create-store.js 98.48% <0.00%> (-1.52%) ⬇️
src/actions.js 100.00% <0.00%> (ø)
src/helpers.js 100.00% <0.00%> (ø)
src/provider.js 100.00% <0.00%> (ø)
src/constants.js 100.00% <0.00%> (ø)
src/listeners.js 100.00% <0.00%> (ø)
src/create-reducer.js 100.00% <0.00%> (ø)
src/effects.js
src/use-local-store.js
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jmyrland
Copy link
Collaborator

Hi! I'm a new collaborator for easy peasy, and I'm currently going through open github issues & PRs.

I like this change! Are you able to resolve the conflicts?

Updated the "Using typed injections" of the typescript tutorial to provide a more tightly bound example of typing the store injections. I believe this is more type safe than the original in the sense that there's now a single source of truth.
@vercel
Copy link

vercel bot commented Sep 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
easy-peasy ✅ Ready (Inspect) Visit Preview Sep 24, 2022 at 9:43AM (UTC)

@jmyrland jmyrland merged commit 9633f18 into ctrlplusb:master Sep 24, 2022
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