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

updates:dashboard #352

Merged
merged 1 commit into from
Apr 15, 2019
Merged

Conversation

matthewcarleton
Copy link
Contributor

just a simple one to test :)

@kubevirt-bot kubevirt-bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/S and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Apr 12, 2019
@coveralls
Copy link

coveralls commented Apr 12, 2019

Pull Request Test Coverage Report for Build 1487

  • 19 of 19 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 87.302%

Totals Coverage Status
Change from base Build 1253: 0.02%
Covered Lines: 3415
Relevant Lines: 3748

💛 - Coveralls

text-align: center;
max-height: 130px;
/* stylelint-disable plugin/selector-bem-pattern */
.kubevirt-cluster-overview__health-grid {
Copy link
Contributor

Choose a reason for hiding this comment

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

can we update StorageOverview/ocs-health.scss for consitency? or unite these two together into one file?

Copy link
Contributor

@rawagner rawagner Apr 15, 2019

Choose a reason for hiding this comment

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

@matthewcarleton I've updated snapshots etc to make the build pass. As @suomiy said, can you please update ocs-health.scss in this PR too ? For other changes I'd suggest to create a new branch and new PR to divide style changes to smaller pieces.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I can do that. Thanks @suomiy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry this got larger than it should have. I'll create a new PR for the following updates.

@kubevirt-bot kubevirt-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 15, 2019
@kubevirt-bot kubevirt-bot added size/XXL and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L labels Apr 15, 2019
@rawagner rawagner merged commit 9b70da1 into kubevirt:master Apr 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants