Skip to content

Re-generate library using /synth.py#247

Closed
dpebot wants to merge 6 commits intomasterfrom
autosynth
Closed

Re-generate library using /synth.py#247
dpebot wants to merge 6 commits intomasterfrom
autosynth

Conversation

@dpebot
Copy link
Contributor

@dpebot dpebot commented Jul 10, 2018

This PR was created by autosynth.

@ghost ghost assigned dpebot Jul 10, 2018
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 10, 2018
@dpebot dpebot force-pushed the autosynth branch 2 times, most recently from ebd43e0 to e94fa20 Compare July 11, 2018 17:08
@codecov
Copy link

codecov bot commented Jul 11, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@4bfd84e). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             master   #247   +/-   ##
=======================================
  Coverage          ?   100%           
=======================================
  Files             ?     10           
  Lines             ?   1266           
  Branches          ?      0           
=======================================
  Hits              ?   1266           
  Misses            ?      0           
  Partials          ?      0

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 4bfd84e...67e37c6. Read the comment docs.

@ghost ghost assigned sduskis Jul 23, 2018
@sduskis
Copy link
Contributor

sduskis commented Jul 24, 2018

@JustinBeckwith, synth produced results that were not approved by the linter. Is that something we ought to be concerned about?

@sduskis sduskis deleted the autosynth branch July 24, 2018 17:17
@JustinBeckwith
Copy link
Contributor

Yes :) I think this was a bug from a few weeks ago that we resolved. autosynth won't open a new PR unless the previous one is merged/closed, so I'm hoping that just closing this causes an update tomorrow that works 🤞

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

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants