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

test: Exclude auto generated go code from the coverage #816

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

test: Exclude auto generated go code from the coverage #816

wants to merge 9 commits into from

Conversation

helayoty
Copy link
Collaborator

Reason for Change:

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Notes for Reviewers:

@codecov Codecov
Copy link

codecov bot commented Jan 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

see 20 files with indirect coverage changes

@helayoty
change file permission
06ca803 
Signed-off-by: Heba Elayoty <[email protected]>
@helayoty
remove script
a4681ce 
Signed-off-by: Heba Elayoty <[email protected]>
@helayoty helayoty force-pushed the helayoty/exclude-code-coverage branch from 1ca72f1 to 6898634 Compare January 10, 2025 07:47
@helayoty helayoty force-pushed the helayoty/exclude-code-coverage branch from 6898634 to b4c1e1d Compare January 10, 2025 07:53
@helayoty
fix python packages
3f45827 
Signed-off-by: Heba Elayoty <[email protected]>
@helayoty helayoty force-pushed the helayoty/exclude-code-coverage branch from b4c1e1d to 3f45827 Compare January 10, 2025 07:58
@helayoty
remove codecov file
d627dfc 
Signed-off-by: Heba Elayoty <[email protected]>
@helayoty
fix python packages
91c75af 
Signed-off-by: Heba Elayoty <[email protected]>
@helayoty helayoty marked this pull request as draft January 10, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fei-Guo Fei-Guo Awaiting requested review from Fei-Guo Fei-Guo is a code owner

@ishaansehgal99 ishaansehgal99 Awaiting requested review from ishaansehgal99 ishaansehgal99 is a code owner

@zhuangqh zhuangqh Awaiting requested review from zhuangqh zhuangqh is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@helayoty