-
Notifications
You must be signed in to change notification settings - Fork 412
chore(storage): Remove storage::internal::CommonMetadata
and storage::internal::AccessControlCommon
#15138
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
base: prepare-for-v3.0.0
Are you sure you want to change the base?
chore(storage): Remove storage::internal::CommonMetadata
and storage::internal::AccessControlCommon
#15138
Conversation
storage::internal::CommonMetadata
and storage::internal::AccessControlCommon
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15138 +/- ##
==========================================
- Coverage 92.93% 92.93% -0.01%
==========================================
Files 2392 2388 -4
Lines 215105 215044 -61
==========================================
- Hits 199904 199847 -57
+ Misses 15201 15197 -4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
storage::internal::CommonMetadata
and storage::internal::AccessControlCommon
storage::internal::CommonMetadata
and storage::internal::AccessControlCommon
This looks good, but we will need to wait to merge it until we're working on v3 due to the potential breaking changes. |
@scotthart do we have a '3.x staging' branch we can point this at? |
Repointed this at the |
45113de
to
14dd653
Compare
Remove
storage::internal::CommonMetadata
andstorage::internal::AccessControlCommon
,fixes #9897
This change is