-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Remove unused code in acceptance tests Sharing.php #31695
Remove unused code in acceptance tests Sharing.php #31695
Conversation
Commit added to backport PR #31694 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I forgot to remove those.
Codecov Report
@@ Coverage Diff @@
## master #31695 +/- ##
=========================================
Coverage 62.89% 62.89%
Complexity 18418 18418
=========================================
Files 1154 1154
Lines 69157 69157
Branches 1260 1260
=========================================
Hits 43499 43499
Misses 25289 25289
Partials 369 369
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If CI passes 👍
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
Remove bonus unused code.
Related Issue
#31531
Motivation and Context
Noticed an unused line of code introduced in #31674
How Has This Been Tested?
CI knows
Types of changes
Checklist: