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

feat: remove YurtAppOverrider #2280

Conversation

techworldhello
Copy link
Contributor

@techworldhello techworldhello commented Jan 27, 2025

What type of PR is this?

/kind feature

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #2263

Special notes for your reviewer:

/assign @rambohe-ch

Does this PR introduce a user-facing change?

NONE

other Note

@techworldhello techworldhello force-pushed the wangjessi/remove-yurtappoverrider branch from 375be1d to 1953469 Compare January 28, 2025 02:21
@techworldhello techworldhello marked this pull request as ready for review January 28, 2025 02:22
@techworldhello techworldhello requested a review from a team as a code owner January 28, 2025 02:22
@techworldhello techworldhello changed the title Remove YurtAppOverrider feat: remove YurtAppOverrider Jan 28, 2025
@rambohe-ch
Copy link
Member

@techworldhello please resolve the conflicted files. maybe you need to rebase the current master branch.

@techworldhello techworldhello force-pushed the wangjessi/remove-yurtappoverrider branch from 1953469 to ebc411d Compare February 4, 2025 00:38
@techworldhello techworldhello force-pushed the wangjessi/remove-yurtappoverrider branch from ebc411d to 9760c31 Compare February 4, 2025 00:50
Copy link

sonarqubecloud bot commented Feb 4, 2025

Copy link

codecov bot commented Feb 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.68%. Comparing base (4647c09) to head (9760c31).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2280      +/-   ##
==========================================
- Coverage   45.85%   45.68%   -0.17%     
==========================================
  Files         418      408      -10     
  Lines       27450    27146     -304     
==========================================
- Hits        12586    12403     -183     
+ Misses      13611    13518      -93     
+ Partials     1253     1225      -28     
Flag Coverage Δ
unittests 45.68% <100.00%> (-0.17%) ⬇️

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

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

@rambohe-ch rambohe-ch added approved approved lgtm lgtm labels Feb 4, 2025
@rambohe-ch
Copy link
Member

/LGTM

@rambohe-ch rambohe-ch merged commit 0e6c0d4 into openyurtio:master Feb 4, 2025
14 checks passed
@techworldhello techworldhello deleted the wangjessi/remove-yurtappoverrider branch February 4, 2025 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved approved lgtm lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature request]remove YurtAppOverrider from OpenYurt
2 participants