Skip to content

Conversation

@dsherry
Copy link
Contributor

@dsherry dsherry commented Feb 16, 2021

No description provided.

@dsherry dsherry added blocker An issue blocking a release. testing Issues related to testing. labels Feb 16, 2021
@codecov
Copy link

codecov bot commented Feb 16, 2021

Codecov Report

Merging #1847 (52475bf) into main (0a57a4b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1847   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         255      255           
  Lines       20658    20658           
=======================================
  Hits        20650    20650           
  Misses          8        8           

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 0a57a4b...52475bf. Read the comment docs.

Copy link
Contributor

@freddyaboulton freddyaboulton left a comment

Choose a reason for hiding this comment

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

@dsherry This is good to merge once CI passes!

Copy link
Contributor

@angela97lin angela97lin left a comment

Choose a reason for hiding this comment

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

👍

We should also file an issue to track uncapping?

@angela97lin
Copy link
Contributor

@dsherry Filed #1848 to track uncapping!

@chukarsten chukarsten mentioned this pull request Feb 23, 2021
@dsherry dsherry mentioned this pull request Mar 10, 2021
@freddyaboulton freddyaboulton deleted the fix_graphviz_windows branch May 13, 2022 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocker An issue blocking a release. testing Issues related to testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants