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.

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

Add coverage #499

Merged
merged 15 commits into from
Aug 12, 2022
Merged

Add coverage #499

Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
e713b79
Add coverage report upload to pre_merge
ashwinvaidya17 Aug 2, 2022
af54087
Use python uploader
ashwinvaidya17 Aug 12, 2022
c82b8e5
Explicitly pass commit hash
ashwinvaidya17 Aug 12, 2022
c128a88
Get ref from expression
ashwinvaidya17 Aug 12, 2022
97715c4
Fix typo
ashwinvaidya17 Aug 12, 2022
fc4e1c8
Debug
ashwinvaidya17 Aug 12, 2022
8eb5302
Merge branch 'main' into ci/add_coverage
ashwinvaidya17 Aug 12, 2022
9d6dc1c
Revert
ashwinvaidya17 Aug 12, 2022
acc6e26
Debug
ashwinvaidya17 Aug 12, 2022
8565077
Debug
ashwinvaidya17 Aug 12, 2022
b223c59
Debug
ashwinvaidya17 Aug 12, 2022
a017dea
Debug
ashwinvaidya17 Aug 12, 2022
fdaf79d
Use variable for commit id
ashwinvaidya17 Aug 12, 2022
856a823
Add coverage badge
ashwinvaidya17 Aug 12, 2022
acaefce
Add comments
ashwinvaidya17 Aug 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 11 additions & 5 deletions .github/workflows/pre_merge.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,14 @@ jobs:
export ANOMALIB_DATASET_PATH=/media/data1/datasets/
export CUDA_VISIBLE_DEVICES=3
tox -e pre_merge
- name: Upload coverage result
uses: actions/upload-artifact@v2
with:
name: coverage
path: .tox/coverage.xml
- name: Upload coverage report
run: |

Copy link
Contributor

Choose a reason for hiding this comment

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

It might be an idea to explain this if-block with a comment for future reference.

if [[ ${{ github.event.pull_request.head.sha }} ]]
then
COMMIT_ID=${{ github.event.pull_request.head.sha }}
else
COMMIT_ID=${{ github.sha }}
fi
pip install codacy-coverage
python-codacy-coverage -r .tox/coverage.xml -t ${{ secrets.CODACY }} -c $COMMIT_ID
1 change: 1 addition & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
[![black](https://img.shields.io/badge/code%20style-black-000000.svg)]()
[![Nightly-Regression Test](https://github.com/openvinotoolkit/anomalib/actions/workflows/nightly.yml/badge.svg)](https://github.com/openvinotoolkit/anomalib/actions/workflows/nightly.yml)
[![Pre-Merge Checks](https://github.com/openvinotoolkit/anomalib/actions/workflows/pre_merge.yml/badge.svg)](https://github.com/openvinotoolkit/anomalib/actions/workflows/pre_merge.yml)
[![Codacy Badge](https://app.codacy.com/project/badge/Coverage/684927c1c76c4c5e94bb53480812fbbb)](https://www.codacy.com/gh/openvinotoolkit/anomalib/dashboard?utm_source=github.com&utm_medium=referral&utm_content=openvinotoolkit/anomalib&utm_campaign=Badge_Coverage)
[![Docs](https://github.com/openvinotoolkit/anomalib/actions/workflows/docs.yml/badge.svg)](https://github.com/openvinotoolkit/anomalib/actions/workflows/docs.yml)
[![Downloads](https://static.pepy.tech/personalized-badge/anomalib?period=total&units=international_system&left_color=grey&right_color=green&left_text=PyPI%20Downloads)](https://pepy.tech/project/anomalib)

Expand Down