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

Revert "OWNERS: Auto Sync OWNERS files from community membership" #2308

Merged
merged 2 commits into from
Nov 9, 2023

Conversation

kaaaaaaang
Copy link
Collaborator

Reverts #2263

Copy link
Contributor

ti-chi-bot bot commented Nov 9, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from kaaaaaaang. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 9, 2023
@codecov-commenter
Copy link

codecov-commenter commented Nov 9, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ef3382f) 50.29% compared to head (2615de1) 48.91%.
Report is 1 commits behind head on master.

❗ Current head 2615de1 differs from pull request most recent head 9e5bc86. Consider uploading reports for the commit 9e5bc86 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2308      +/-   ##
==========================================
- Coverage   50.29%   48.91%   -1.38%     
==========================================
  Files         324      325       +1     
  Lines       34693    34704      +11     
==========================================
- Hits        17446    16972     -474     
- Misses      14962    15671     +709     
+ Partials     2285     2061     -224     
Flag Coverage Δ
cluster 34.42% <ø> (-10.40%) ⬇️
dm 25.26% <ø> (-0.21%) ⬇️
tiup 15.56% <ø> (ø)
unittest 22.33% <ø> (?)

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

see 124 files with indirect coverage changes

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

@kaaaaaaang kaaaaaaang merged commit 1de0e18 into master Nov 9, 2023
7 of 16 checks passed
@kaaaaaaang kaaaaaaang deleted the revert-2263-bot/update-owners-1692844199071 branch November 9, 2023 09:21
@kaaaaaaang kaaaaaaang added this to the v1.14.0 milestone Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants