Skip to content

fix(types): make SkeletonProps.rows optional to match the docs#5179

Merged
chasestarr merged 3 commits intouber:masterfrom
alexgorbatchev:fix-skeletonprops-rows
Oct 4, 2022
Merged

fix(types): make SkeletonProps.rows optional to match the docs#5179
chasestarr merged 3 commits intouber:masterfrom
alexgorbatchev:fix-skeletonprops-rows

Conversation

@alexgorbatchev
Copy link
Contributor

Description

According to the docs SkeletonProps.rows is optional, however it's not so according to the typings. This PR makes SkeletonProps.rows optional.

Scope

Patch: Bug Fix

@CLAassistant
Copy link

CLAassistant commented Sep 30, 2022

CLA assistant check
All committers have signed the CLA.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 30, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d33a3c8:

Sandbox Source
Basic usage Configuration

@alexgorbatchev alexgorbatchev changed the title fix(skeleton): make SkeletonProps.rows optional to match the docs fix(typings): make SkeletonProps.rows optional to match the docs Sep 30, 2022
@alexgorbatchev alexgorbatchev changed the title fix(typings): make SkeletonProps.rows optional to match the docs fix(types): make SkeletonProps.rows optional to match the docs Sep 30, 2022
@alexgorbatchev alexgorbatchev force-pushed the fix-skeletonprops-rows branch from 1916030 to 73784b1 Compare October 4, 2022 16:11
Copy link
Collaborator

@chasestarr chasestarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you

@chasestarr chasestarr merged commit c5b0e46 into uber:master Oct 4, 2022
@alexgorbatchev alexgorbatchev deleted the fix-skeletonprops-rows branch October 6, 2022 23:26
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.

3 participants