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

Mccalluc/provvis regression #2223

Merged
merged 2 commits into from Oct 12, 2017
Merged

Mccalluc/provvis regression #2223

merged 2 commits into from Oct 12, 2017

Conversation

mccalluc
Copy link
Member

I went a lot farther down this path, but then realized that my edits weren't actually being deployed. In any event, I'm pretty much flying blind here: There is so much functionality behind this, and my only test is whether a rectangle shows up on the screen, and even that is not automated.

@codecov-io
Copy link

Codecov Report

Merging #2223 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2223   +/-   ##
========================================
  Coverage    48.64%   48.64%           
========================================
  Files          412      412           
  Lines        28172    28172           
  Branches      1312     1312           
========================================
  Hits         13704    13704           
  Misses       14468    14468

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 0e612c8...637166d. Read the comment docs.

Copy link
Member

@scottx611x scottx611x left a comment

Choose a reason for hiding this comment

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

Looks good to me. I think I led you down this path in the first place so sorry for that

@scottx611x
Copy link
Member

scottx611x commented Oct 11, 2017

Also you may want to hold off on merging since our outdated version of bower is causing builds to fail. I have a PR right now that will fix it though

@scottx611x scottx611x merged commit 2bdf6f2 into develop Oct 12, 2017
@scottx611x scottx611x deleted the mccalluc/provvis-regression branch October 12, 2017 15:12
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.

None yet

3 participants