Skip to content

fix(TS): make container optional in RenderOptions #253

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

Merged
merged 1 commit into from
Dec 20, 2018

Conversation

kentcdodds
Copy link
Member

What: TS: make container optional in RenderOptions

Why: Particularly useful if you're making a custom render with custom options. If you extend the type and call your custom render without the container but with the other options then you get a false error.

How: ?

Checklist:

  • Documentation N/A
  • Tests N/A
  • Ready to be merged
  • Added myself to contributors table N/A

Particularly useful if you're making a custom render with custom options.
@codecov
Copy link

codecov bot commented Dec 20, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #253   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          27     27           
  Branches        5      5           
=====================================
  Hits           27     27

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 f099d2f...fad17f2. Read the comment docs.

@kentcdodds kentcdodds merged commit 4763ffb into master Dec 20, 2018
@kentcdodds kentcdodds deleted the pr/ts-render-options branch December 20, 2018 19:26
@kentcdodds
Copy link
Member Author

Hey friends 👋 look I'm using TypeScript 🎉

@kentcdodds
Copy link
Member Author

🎉 This PR is included in version 5.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant