Skip to content

Conversation

@pablogsal
Copy link

No description provided.

@lysnikolaou
Copy link
Member

This currently builds the Docs and runs the Docs tests on Travis. Shouldn't we deactivate those or Travis entirely?

Copy link

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

Looks like it works!

@pablogsal
Copy link
Author

Shouldn't we deactivate those or Travis entirely?

Right, I assume we can only GitHub actions.

@gvanrossum
Copy link

Is this PR against master or pegen?

@lysnikolaou
Copy link
Member

Is this PR against master or pegen?

Pegen.

@pablogsal pablogsal force-pushed the ci branch 3 times, most recently from 7a4ad22 to 820b193 Compare February 26, 2020 01:42
@pablogsal
Copy link
Author

Ok, removed Travis and docs.

@codecov-io
Copy link

codecov-io commented Feb 26, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##             pegen       #3   +/-   ##
========================================
  Coverage         ?   82.11%           
========================================
  Files            ?     1955           
  Lines            ?   584171           
  Branches         ?    44464           
========================================
  Hits             ?   479716           
  Misses           ?    94812           
  Partials         ?     9643

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 d644891...e2e11ea. Read the comment docs.

@pablogsal pablogsal merged commit 8474234 into pegen Feb 26, 2020
@pablogsal pablogsal deleted the ci branch February 26, 2020 02:03
gvanrossum pushed a commit that referenced this pull request Mar 18, 2020
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.

5 participants