Skip to content
This repository was archived by the owner on May 17, 2019. It is now read-only.

Use required context parameter for InitialStateType #132

Merged
merged 3 commits into from
Jun 27, 2018

Conversation

derekjuber
Copy link
Contributor

This fixes a potential flow warning if developers attempt to use a custom method for GetInitialStateToken.

Also fixes the documentation to reflect this required parameter.

@derekjuber derekjuber requested a review from AlexMSmithCA June 27, 2018 22:26
@codecov
Copy link

codecov bot commented Jun 27, 2018

Codecov Report

Merging #132 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #132   +/-   ##
=======================================
  Coverage   94.64%   94.64%           
=======================================
  Files           5        5           
  Lines          56       56           
  Branches       12       12           
=======================================
  Hits           53       53           
  Misses          1        1           
  Partials        2        2
Impacted Files Coverage Δ
src/tokens.js 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b2af7b...7726629. Read the comment docs.

@derekjuber
Copy link
Contributor Author

!merge

@old-fusion-bot old-fusion-bot bot merged commit 095072a into fusionjs:master Jun 27, 2018
@derekjuber derekjuber deleted the use_required_type branch July 16, 2018 20:32
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.

3 participants