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

Document how to access typings πŸ•΅πŸ»β€β™‚οΈ #253

Merged
merged 1 commit into from
Mar 12, 2020

Conversation

ubbe-xyz
Copy link
Contributor

@ubbe-xyz ubbe-xyz commented Mar 11, 2020

Summary

Our types are not exported in the root of the package, given there's an issue with Webpack.

They're exported under @zopauk/react-components/types, document that given people might be easily confused around it πŸ‘πŸ»

Issue

#212

@codecov
Copy link

codecov bot commented Mar 11, 2020

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   92.71%   92.71%           
=======================================
  Files          70       70           
  Lines         645      645           
  Branches      124      178   +54     
=======================================
  Hits          598      598           
  Misses         47       47

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 128f36e...018e140. Read the comment docs.

@ubbe-xyz ubbe-xyz mentioned this pull request Mar 11, 2020
15 tasks
@ubbe-xyz ubbe-xyz merged commit 8819fb4 into master Mar 12, 2020
@ubbe-xyz ubbe-xyz deleted the docs/212-exported-types branch March 12, 2020 16:46
@poteirard
Copy link
Contributor

πŸŽ‰ This PR is included in version 2.12.1 πŸŽ‰

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.

4 participants