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

[BUILD] Bump actions/labeler from v4 to v5 and update labeler.yml #5843

Closed
wants to merge 9 commits into from

Conversation

bowenliang123
Copy link
Contributor

@bowenliang123 bowenliang123 commented Dec 11, 2023

🔍 Description

Issue References 🔗

Describe Your Solution 🔧

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests


Checklists

📝 Author Self Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

  • Pull request title is okay.
  • No license issues.
  • Milestone correctly set?
  • Test coverage is ok
  • Assignees are selected.
  • Minimum number of approvals
  • No changes are requested

Be nice. Be informative.

@github-actions github-actions bot added kind:infra license, community building, project builds, asf infra related, etc. module:server kind:build labels Dec 11, 2023
@bowenliang123
Copy link
Contributor Author

Test with changed pom.xml file in /kyuubi-server, this PR labelled build:server correctly.
image

@codecov-commenter
Copy link

codecov-commenter commented Dec 11, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8ab4763) 61.47% compared to head (adb7b9f) 61.38%.

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

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5843      +/-   ##
============================================
- Coverage     61.47%   61.38%   -0.09%     
  Complexity       23       23              
============================================
  Files           608      608              
  Lines         36159    36159              
  Branches       4958     4958              
============================================
- Hits          22228    22198      -30     
- Misses        11545    11567      +22     
- Partials       2386     2394       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bowenliang123
Copy link
Contributor Author

Don't know why the CI still uses actions/labeler v4 but not the targeted v5.
https://github.com/apache/kyuubi/pull/5843/checks#step:2:1
cc @pan3793

@bowenliang123 bowenliang123 marked this pull request as ready for review December 12, 2023 00:58
@bowenliang123 bowenliang123 marked this pull request as draft December 12, 2023 01:46
@bowenliang123 bowenliang123 deleted the labeler5-any branch December 12, 2023 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:build kind:infra license, community building, project builds, asf infra related, etc. module:server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants