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

fix(popover): maxWidth typescript error #2110

Merged
merged 1 commit into from May 29, 2019

Conversation

@dlabrecq
Copy link
Member

dlabrecq commented May 29, 2019

Fixes #2108

There is already a default value for maxWidth, so we just need to make this optional. Tested with the Cost Management UI.

@patternfly-build

This comment has been minimized.

Copy link
Contributor

patternfly-build commented May 29, 2019

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented May 29, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2110   +/-   ##
=======================================
  Coverage   81.09%   81.09%           
=======================================
  Files         643      643           
  Lines        7706     7706           
  Branches      451      451           
=======================================
  Hits         6249     6249           
  Misses       1257     1257           
  Partials      200      200
Flag Coverage Δ
#patternfly3 84.88% <ø> (ø) ⬆️
#patternfly4 76.89% <ø> (ø) ⬆️
#patternflymisc 95.68% <ø> (ø) ⬆️
Impacted Files Coverage Δ
...ly-4/react-core/src/components/Popover/Popover.tsx 61.53% <ø> (ø) ⬆️

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 6c04803...f86f301. Read the comment docs.

@redallen redallen merged commit e41ae79 into patternfly:master May 29, 2019
2 checks passed
2 checks passed
ci/circleci: build Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@dlabrecq dlabrecq deleted the dlabrecq:2018-popover-ts branch Jun 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.