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

[improve][ci] Allow [fix][broker][branch-2.8] alike patten #18504

Merged
merged 2 commits into from
Nov 17, 2022

Conversation

tisonkun
Copy link
Member

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Nov 16, 2022
@tisonkun tisonkun changed the title [improve][ci] Allow [fix][broker][branch-2.9] alike patten [improve][ci] Allow [fix][broker][branch-2.8] alike patten Nov 16, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 16, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.30%. Comparing base (7975023) to head (0f1bb2c).
Report is 2287 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #18504      +/-   ##
============================================
+ Coverage     45.62%   47.30%   +1.68%     
- Complexity    10075    10460     +385     
============================================
  Files           697      697              
  Lines         68024    68015       -9     
  Branches       7293     7285       -8     
============================================
+ Hits          31033    32176    +1143     
+ Misses        33413    32245    -1168     
- Partials       3578     3594      +16     
Flag Coverage Δ
unittests 47.30% <ø> (+1.68%) ⬆️

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

see 73 files with indirect coverage changes

@tisonkun
Copy link
Member Author

Merging...

Verified on https://regex101.com/

@tisonkun tisonkun merged commit 55d7deb into apache:master Nov 17, 2022
@tisonkun tisonkun deleted the better-semantic-pr branch November 17, 2022 00:35
@@ -61,8 +61,8 @@ jobs:
# bk -> bookkeeper
scopes: |
admin
bk
Copy link
Member Author

@tisonkun tisonkun Nov 17, 2022

Choose a reason for hiding this comment

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

@michaeljmarshall I find that you add bk and zk scopes. There're several overlapping scopes now:

  • zk vs meta
  • bk vs ml vs storage

How do you understand this situation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-not-needed Your PR changes do not impact docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants