-
Notifications
You must be signed in to change notification settings - Fork 835
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(instrumentation): re-export initialize from import-in-the-middle #5123
feat(instrumentation): re-export initialize from import-in-the-middle #5123
Conversation
16803e7
to
e883ea8
Compare
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.
LGTM, but for the copyright change.
@cjpearson Oh, also, are you able to sign the CLA (see the earlier comment about that)? |
e883ea8
to
e1e791a
Compare
Signed-off-by: Connor Pearson <connor.pearson@mailbox.org>
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.
thanks 👍
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
not stale |
outdated, copyright change has been reverted
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5123 +/- ##
==========================================
- Coverage 94.58% 94.57% -0.02%
==========================================
Files 318 318
Lines 8069 8069
Branches 1701 1701
==========================================
- Hits 7632 7631 -1
- Misses 437 438 +1 |
Which problem is this PR solving?
This re-exports
initialize
fromimport-in-the-middle
to support the including or excluding of specific modules.Fixes #5121
Short description of the changes
Imports and re-exports
initialize
fromimport-in-the-middle/hook.mjs
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
import-in-the-middle
that the provided options are presentChecklist: