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

Upload coverage from extension tests to Codecov #6781

Merged
merged 1 commit into from
Jul 1, 2022

Conversation

jwodder
Copy link
Member

@jwodder jwodder commented Jun 30, 2022

Closes #6780.

Changelog

🛡 Tests

  • Upload coverage from extension tests to Codecov

@jwodder jwodder added tests Add or improve existing tests semver-tests Changes only affect tests, no impact on version and removed tests Add or improve existing tests labels Jun 30, 2022
@@ -38,7 +38,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r requirements.txt
pip install -r requirements-devel.txt
Copy link
Member

Choose a reason for hiding this comment

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

note -- the difference between the two besides [full] and [devel] flavors of install is that in -devel we install with pip -e i.e. for development. Should be ok, but just worth noting.

Copy link
Member Author

Choose a reason for hiding this comment

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

Note that I changed it to -devel in order to get coverage installed.

@yarikoptic
Copy link
Member

Just FTR - seems have successfully submitted for e.g. metalad extension... let's wait to see if coverage report comes up with some "extra coverage"
Run codecov/codecov-action@v3
==> linux OS detected
https://uploader.codecov.io/latest/linux/codecov.SHA256SUM
==> SHASUM file signed by key id 806bb28aed[7](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:8)79[8](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:9)6[9](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:10)
==> Uploader SHASUM verified (432553f03714a83ab5cac78be2e7fe6a6050767f78f77e9c7a84bdb1c34a0bd3  codecov)
==> Running version latest
==> Running version v0.2.4
/home/runner/work/_actions/codecov/codecov-action/v3/dist/codecov -n  -Q github-action-3.1.0 -C 1bc760f0cfa1d139fd57941d618fed344892450d -s __testhome__
[2022-06-30T19:19:35.393Z] ['info'] 
     _____          _
    / ____|        | |
   | |     ___   __| | ___  ___ _____   __
   | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
   | |___| (_) | (_| |  __/ (_| (_) \ V /
    \_____\___/ \__,_|\___|\___\___/ \_/
  Codecov report uploader 0.2.4
[2022-06-30T19:19:35.401Z] ['info'] => Project root located at: /home/runner/work/datalad/datalad
[2022-06-30T19:19:35.403Z] ['info'] -> No token specified or token is empty
[2022-06-30T19:19:35.556Z] ['info'] Searching for coverage files...
[2022-06-30T19:19:35.615Z] ['info'] Warning: Some files located via search were excluded from upload.
[2022-06-30T19:19:35.616Z] ['info'] If Codecov did not locate your files, please review https://docs.codecov.com/docs/supported-report-formats
[2022-06-30T19:19:35.616Z] ['info'] => Found 1 possible coverage files:
  coverage.xml
[2022-06-30T19:19:35.617Z] ['info'] Processing __testhome__/coverage.xml...
[2022-06-30T19:19:35.652Z] ['info'] Detected GitHub Actions as the CI provider.
[2022-06-30T19:19:35.656Z] ['info'] Pinging Codecov: https://codecov.io/upload/v4?package=github-action-3.1.0-uploader-0.2.4&token=*******&branch=gh-6780&build=2592184996&build_url=https%3A%2F%2Fgithub.com%2Fdatalad%2Fdatalad%2Factions%2Fruns%2F2592184996&commit=1bc760f0cfa1d139fd57941d618fed344892450d&job=Extensions&pr=6781&service=github-actions&slug=datalad%2Fdatalad&name=&tag=&flags=&parent=
[2022-06-30T19:19:36.186Z] ['info'] https://codecov.io/github/datalad/datalad/commit/1bc760f0cfa1d139fd57941d618fed344892450d
https://storage.googleapis.com/codecov/v4/raw/2022-06-30/295CAAAFF4A547D0B795AB8A4785AE7C/1bc760f0cfa1d139fd57941d618fed344892450d/12262788-66ee-4453-a787-7352f3a91e4a.txt?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=GOOG1EKKHVKCKHW7KBCGM7IHR55T63V2PAVJWLVFNITJHDU5G6R5IRN3LMWJA%2F20220630%2FUS%2Fs3%2Faws4_request&X-Amz-Date=20220630T191936Z&X-Amz-Expires=[10](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:11)&X-Amz-SignedHeaders=host&X-Amz-Signature=a34d40f0a8c0ac4de7d3ce41058b6403c4527bf7790a6589e86621642c7bacfe
[2022-06-30T19:19:36.188Z] ['info'] Uploading...
[2022-06-30T19:19:36.398Z] ['info'] {"status":"success","resultURL":"https://codecov.io/github/datalad/datalad/commit/1bc760f0cfa1d[13](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:14)9fd57941d6[18](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:19)fed34489[24](https://github.com/datalad/datalad/runs/7138123353?check_suite_focus=true#step:13:25)50d"}

@yarikoptic
Copy link
Member

mac fails with recent annex on appveyor are known. I am surprised that there is no codecov report but I don't see how it could relate to these changes. So, I will just proceed with merging since changes seems "to do the right thing" and otherwise non-consequential. Thank you @jwodder !

@github-actions
Copy link

github-actions bot commented Jul 6, 2022

🚀 PR was released in 0.16.7 🚀

@jwodder jwodder deleted the gh-6780 branch May 12, 2023 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released semver-tests Changes only affect tests, no impact on version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants