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

[ci] Refactor backend Github Actions workflow and add pytest coverage PR comment #4028

Merged
merged 8 commits into from
Mar 6, 2025
Merged

[ci] Refactor backend Github Actions workflow and add pytest coverage PR comment #4028

merged 8 commits into from
Mar 6, 2025

Conversation

Harshg999
Copy link
Collaborator

What changes were proposed in this pull request?

  • Add code coverage numbers on PR as comment

How was this patch tested?

  • Check using Github Actions on PR

@cloudera cloudera deleted a comment from github-actions bot Mar 6, 2025
@cloudera cloudera deleted a comment from github-actions bot Mar 6, 2025
@cloudera cloudera deleted a comment from github-actions bot Mar 6, 2025
@cloudera cloudera deleted a comment from github-actions bot Mar 6, 2025
@Harshg999 Harshg999 self-assigned this Mar 6, 2025
@Harshg999 Harshg999 changed the title [ci] Add pytest coverage comment action and configure junit XML output [ci] Refactor backend Github Actions workflow and add pytest coverage PR comment Mar 6, 2025
@Harshg999 Harshg999 enabled auto-merge (squash) March 6, 2025 10:54
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 6, 2025 •
edited
Loading

Backend Codecov

Backend Code Coverage Report •
FileStmtsMissCoverMissing
TOTAL533172613850% 
report-only-changed-files is enabled. No files were changed during this commit :)

Pytest Report

Tests Skipped Failures Errors Time
1088 107 💤 0 ❌ 0 🔥 5m 0s ⏱️

ramprasadagarwal
ramprasadagarwal approved these changes Mar 6, 2025
View reviewed changes
Copy link
Collaborator

@ramprasadagarwal ramprasadagarwal left a comment

Choose a reason for hiding this comment

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

Nice work!

bjornalm
bjornalm approved these changes Mar 6, 2025
View reviewed changes
Copy link
Collaborator

@bjornalm bjornalm left a comment

Choose a reason for hiding this comment

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

Nice one!

@Harshg999 Harshg999 merged commit ba0afec into master Mar 6, 2025
6 checks passed
@Harshg999 Harshg999 deleted the add-pr-comment branch March 6, 2025 13:25
athithyaaselvam pushed a commit that referenced this pull request Mar 11, 2025
@Harshg999
[ci] Refactor backend Github Actions workflow and add pytest coverage…
0fbcc38 
… PR comment (#4028)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bjornalm bjornalm bjornalm approved these changes

@ramprasadagarwal ramprasadagarwal ramprasadagarwal approved these changes

@wing2fly wing2fly Awaiting requested review from wing2fly

@amitsrivastava amitsrivastava Awaiting requested review from amitsrivastava

@tabraiz12 tabraiz12 Awaiting requested review from tabraiz12

@JohanAhlen JohanAhlen Awaiting requested review from JohanAhlen

@ranade1 ranade1 Awaiting requested review from ranade1

@athithyaaselvam athithyaaselvam Awaiting requested review from athithyaaselvam

@agl29 agl29 Awaiting requested review from agl29

Assignees

@Harshg999 Harshg999

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Harshg999 @bjornalm @ramprasadagarwal