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

Add GitHub Actions #128

Closed
wants to merge 4 commits into from
Closed

Add GitHub Actions #128

wants to merge 4 commits into from

Conversation

eumiro
Copy link
Contributor

@eumiro eumiro commented Jan 18, 2021

Let's see whether this is a good replacement for the current .travis.yml file…

@codecov-io
Copy link

codecov-io commented Jan 18, 2021

Codecov Report

Merging #128 (fa78071) into master (fba09fa) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #128   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          466       466           
=========================================
  Hits           466       466           

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 fba09fa...fa78071. Read the comment docs.

@eumiro eumiro marked this pull request as ready for review January 18, 2021 21:46
@eumiro
Copy link
Contributor Author

eumiro commented Jan 18, 2021

Since the master branch does not contain GHA until now, we cannot see the results of the GHA CI run here.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Show resolved Hide resolved
@xflr6
Copy link
Owner

xflr6 commented Feb 14, 2021

Thanks, left some comments.,

@eumiro
Copy link
Contributor Author

eumiro commented Feb 14, 2021

BTW, the results of this PR can be found at https://github.com/eumiro/graphviz/actions

@eumiro
Copy link
Contributor Author

eumiro commented Feb 14, 2021

Added a README.rst badge for the GHA CI. Please squash the commits afterwards.

xflr6 added a commit that referenced this pull request Jul 11, 2021
@xflr6
Copy link
Owner

xflr6 commented Jul 11, 2021

Sorry for the delay.

Still prefer not using tox in the CI and want to continue collecting coverage from all jobs (to cover for compat code branches).

Superseded by a0668ea and 78885a8, closing, thanks for the PR.

@xflr6 xflr6 closed this Jul 11, 2021
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.

3 participants