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

chore(*): add pr status check to CI pipeline #41

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

caohe
Copy link
Member

@caohe caohe commented Apr 21, 2023

What type of PR is this?

Chore

What this PR does / why we need it:

Add PR status checks to the CI pipeline.

Which issue(s) this PR fixes:

Special notes for your reviewer:

@caohe caohe added chore workflow/draft draft: no need to review labels Apr 21, 2023
@caohe caohe added this to the v0.2 milestone Apr 21, 2023
@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.07 ⚠️

Comparison is base (9abc2bc) 51.02% compared to head (cc270c1) 50.95%.

❗ Current head cc270c1 differs from pull request most recent head a1cdeb2. Consider uploading reports for the commit a1cdeb2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
- Coverage   51.02%   50.95%   -0.07%     
==========================================
  Files         317      311       -6     
  Lines       32323    31776     -547     
==========================================
- Hits        16494    16193     -301     
+ Misses      13894    13668     -226     
+ Partials     1935     1915      -20     
Flag Coverage Δ
unittest 50.95% <ø> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 51 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@caohe caohe added workflow/merge-hold merge-hold: code is ready but still has dependency and removed workflow/merge-hold merge-hold: code is ready but still has dependency labels Apr 21, 2023
@caohe caohe added workflow/merge-hold merge-hold: code is ready but still has dependency and removed workflow/merge-hold merge-hold: code is ready but still has dependency labels Apr 27, 2023
@caohe caohe force-pushed the block-hold-pr branch 2 times, most recently from 15d0eda to 1f5fe1e Compare April 27, 2023 12:52
@caohe caohe added workflow/merge-hold merge-hold: code is ready but still has dependency and removed workflow/merge-hold merge-hold: code is ready but still has dependency labels Apr 27, 2023
@caohe caohe added workflow/merge-hold merge-hold: code is ready but still has dependency and removed workflow/merge-hold merge-hold: code is ready but still has dependency labels Apr 27, 2023
@caohe caohe marked this pull request as ready for review April 28, 2023 03:44
@caohe caohe added workflow/need-review review: test succeeded, need to review and removed workflow/draft draft: no need to review labels Apr 28, 2023
Signed-off-by: caohe <caohe9603@gmail.com>
@caohe caohe added the workflow/merge-ready merge-ready: code is ready and can be merged label Apr 28, 2023
@waynepeking348 waynepeking348 merged commit 7de3f46 into kubewharf:main Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore workflow/merge-ready merge-ready: code is ready and can be merged workflow/need-review review: test succeeded, need to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants