Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

BREAKING: export all typings #382

Merged
merged 4 commits into from
Oct 22, 2018
Merged

BREAKING: export all typings #382

merged 4 commits into from
Oct 22, 2018

Conversation

levithomason
Copy link
Member

@levithomason levithomason commented Oct 19, 2018

This PR exports all typings at the top level. This required a number of changes, including cleaning up our names and paths so that consumers have a more sane API to work with.

All components were also exported top level in the process, some were missing.

Refactor

  1. removed duplicate src/components/*/index.ts files
  2. exported all components from top-level index file
  3. exported all typings from top-level index file
  4. remove I* prefix form interfaces
  5. replace “Part” naming with “Slot” in typings
  6. Moved theme typings to theme/typings.ts (required to generate def files properly)

Fixes

Fixed menu behavior examples
Fixed a lot of other issues necessary for refactors...

@codecov
Copy link

codecov bot commented Oct 19, 2018

Codecov Report

Merging #382 into master will increase coverage by 0.22%.
The diff coverage is 98.7%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #382      +/-   ##
==========================================
+ Coverage   90.55%   90.77%   +0.22%     
==========================================
  Files          67       41      -26     
  Lines        1334     1334              
  Branches      169      194      +25     
==========================================
+ Hits         1208     1211       +3     
+ Misses        123      120       -3     
  Partials        3        3
Impacted Files Coverage Δ
src/components/Attachment/Attachment.tsx 97.14% <ø> (ø) ⬆️
src/components/Popup/PopupContent.tsx 100% <100%> (ø) ⬆️
src/components/Slot/Slot.tsx 100% <100%> (ø) ⬆️
src/components/Ref/Ref.tsx 100% <100%> (ø) ⬆️
src/index.ts 100% <100%> (ø) ⬆️
src/components/Image/Image.tsx 100% <100%> (ø) ⬆️
src/components/ItemLayout/ItemLayout.tsx 82.6% <100%> (ø) ⬆️
src/components/Segment/Segment.tsx 100% <100%> (ø) ⬆️
src/components/Portal/Portal.tsx 100% <100%> (ø) ⬆️
src/components/Avatar/Avatar.tsx 100% <100%> (ø) ⬆️
... and 30 more

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 7d92b3d...b841270. Read the comment docs.

@levithomason levithomason merged commit c68f814 into master Oct 22, 2018
@levithomason levithomason deleted the feat/export-types branch October 22, 2018 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants