-
Notifications
You must be signed in to change notification settings - Fork 585
chore: rename workspace dirs to packages/${package-base-name} (extreme edition) #2920
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: main
Are you sure you want to change the base?
Conversation
…e edition) This is take 2 of open-telemetry#2223 This implements option 3 of open-telemetry#2223 (comment) (i.e. the "extreme" option)
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2920 +/- ##
=======================================
Coverage 89.76% 89.76%
=======================================
Files 187 187
Lines 9149 9149
Branches 1885 1885
=======================================
Hits 8213 8213
Misses 936 936
🚀 New features to boost your workflow:
|
This package does not have an assigned component owner and is considered unmaintained. As such this package is in feature-freeze and this PR will be closed with 14 days unless a new owner or a sponsor (a member of @open-telemetry/javascript-approvers) for the feature is found. It is the responsibility of the author to find a sponsor for this feature. |
This is take 2 of #2223
This implements option 3 of #2223 (comment)
(i.e. the "extreme" option)
Status
DO NOT MERGE
Let's not merge this yet, even if agreed on doing this. I think we can finish the instr-redis consolidation, and drop the redis-common package first.