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

Fix travis CI badge #371

Merged
merged 1 commit into from May 30, 2019

Conversation

Projects
None yet
2 participants
@Mariatta
Copy link
Member

commented May 30, 2019

It has moved from travis-ci.org to travis-ci.com

What do these changes do?

Update the Travis CI badge in readme. Currently it is not rendered.

Are there changes in behavior for the user?

With this fix, readers can see the proper travis CI badge in readme.

Related issue number

None

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt
    • The format is <Name> <Surname>.
    • Please keep alphabetical order, the file is sorted by names.
  • Add a new news fragment into the CHANGES folder
    • name it <issue_id>.<type> (e.g. 588.bugfix)
    • if you don't have an issue_id change it to the pr id after creating the PR
    • ensure type is one of the following:
      • .feature: Signifying a new feature.
      • .bugfix: Signifying a bug fix.
      • .doc: Signifying a documentation improvement.
      • .removal: Signifying a deprecation or removal of public API.
      • .misc: A ticket has been closed, but it is not of interest to users.
    • Make sure to use full sentences with correct case and punctuation, for example: Fix issue with non-ascii contents in doctest text files.
Fix travis CI badge
It has moved from travis-ci.org to travis-ci.com
@codecov

This comment has been minimized.

Copy link

commented May 30, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #371   +/-   ##
=======================================
  Coverage   97.26%   97.26%           
=======================================
  Files           4        4           
  Lines         219      219           
  Branches       26       26           
=======================================
  Hits          213      213           
  Misses          4        4           
  Partials        2        2

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 10962a6...ba1dab7. Read the comment docs.

@asvetlov asvetlov merged commit 8ad252d into aio-libs:master May 30, 2019

3 of 5 checks passed

LGTM analysis: JavaScript No code changes detected
Details
LGTM analysis: Python No code changes detected
Details
Travis CI - Pull Request Build Passed
Details
codecov/patch Coverage not affected when comparing 10962a6...ba1dab7
Details
codecov/project 97.26% remains the same compared to 10962a6
Details
@asvetlov

This comment has been minimized.

Copy link
Member

commented May 30, 2019

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.