Skip to content

Remove CodeCov support #1105

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.

Sign up for GitHub

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

Jump to bottom
Closed
php-coder opened this issue Jul 27, 2019 · 3 comments
Closed

Remove CodeCov support #1105

php-coder opened this issue Jul 27, 2019 · 3 comments
Assignees
@php-coder
Labels
ADR/needed A decision has been made and it needs to be documented area/build scripts area/infrastructure impact/changelog This change should be reflected in the NEWS.txt file
Milestone
0.4.1

Comments

@php-coder
Copy link
Owner

php-coder commented Jul 27, 2019 •
edited
Loading

The codecov bot doesn't do anything useful to us. Also, we don't care much about code coverage and numbers. JaCoCo is enough. Let's decommission CodeCov.

  • update NEWS.txt
  • update README.md (remove a badge)
  • remove publish_code_coverage.sh + update .travis.yml
  • GitHub: decide whether we need to revoke an access or not
  • GitHub: remove a webhook
  • (probably) disable integration on CodeCov site
@php-coder php-coder added area/infrastructure impact/changelog This change should be reflected in the NEWS.txt file area/build scripts labels Jul 27, 2019
@php-coder php-coder added this to the 0.4.1 milestone Jul 27, 2019
@php-coder php-coder self-assigned this Jul 27, 2019
@php-coder php-coder mentioned this issue Jul 27, 2019
Closed
@php-coder php-coder mentioned this issue Aug 11, 2019
Closed
php-coder added a commit that referenced this issue Aug 11, 2019
@php-coder
ci: cancel code coverage publishing to codecov.io service.
471a871 
The codecov bot doesn't do anything useful to us as we don't care much
about a percentage of code coverage. JaCoCo is enough. So this is why I
have decided to decommission CodeCov.

Addressed to #1105
@php-coder php-coder added in progress ADR/needed A decision has been made and it needs to be documented labels Aug 11, 2019
@php-coder
Copy link
Owner Author

GitHub: decide whether we need to revoke an access or not

Let's keep it as it can't be undone.

@php-coder
Copy link
Owner Author

(probably) disable integration on CodeCov site

A whole account has been deleted.

@php-coder
Copy link
Owner Author

Done.

@php-coder php-coder mentioned this issue Jan 10, 2020
Open
70 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@php-coder php-coder

Labels
ADR/needed A decision has been made and it needs to be documented area/build scripts area/infrastructure impact/changelog This change should be reflected in the NEWS.txt file
Projects
None yet
Milestone
0.4.1
Development

No branches or pull requests
1 participant
@php-coder