-
Couldn't load subscription status.
- Fork 231
Updating PatternFly and Angular-PatternFly to v3.21.0 #1284
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
Conversation
Do we know what's adding so many lines? |
|
@spadgett, it's the new fonts. |
|
You may not be able to remove them completely until we get a new devenv_ami
build. You probably need to just update the resolution version numbers as
part of this PR for now.
…On Tue, Feb 21, 2017 at 4:16 PM, Robb Hamilton ***@***.***> wrote:
@spadgett <https://github.com/spadgett>, it's the new fonts
<patternfly/patternfly@c9800ee>
.
Hold on this one sec as I forgot to remove the resolutions we added to get
KVE to merge.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#1284 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABZk7asmwE279jtRgcAnIkdiXR2Wj6u9ks5re1QZgaJpZM4MH3dR>
.
|
|
Ok. I bumped the resolution version numbers. It should be ok now. |
|
@spadgett, once this merges, can you rebase your overview-redesign branch again since this PR has the necessary list-pf CSS? |
|
Will do |
|
[test] |
|
Evaluated for origin web console test up to 23462f1 |
|
Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1070/) (Base Commit: 36c701f) |
|
[merge] |
|
Evaluated for origin web console merge up to 23462f1 |
|
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1071/) (Base Commit: 5d8765c) |
No description provided.