Skip to content

Conversation

@malfet
Copy link
Contributor

@malfet malfet commented Oct 8, 2020

@malfet malfet requested review from ailzhang and seemethere October 8, 2020 16:56
@malfet malfet force-pushed the malfet/do-not-rebase-nightly branch from 6084bbd to 9ff9c2f Compare October 8, 2020 16:58
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@malfet has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

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

LGTM!

@codecov
Copy link

codecov bot commented Oct 8, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #46038   +/-   ##
=======================================
  Coverage   68.28%   68.28%           
=======================================
  Files         410      410           
  Lines       53306    53306           
=======================================
  Hits        36398    36398           
  Misses      16908    16908           

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 ea4fbb2...9ff9c2f. Read the comment docs.

@malfet malfet deleted the malfet/do-not-rebase-nightly branch October 8, 2020 21:50
@facebook-github-bot
Copy link
Contributor

@malfet merged this pull request in ddcacc7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants