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
fix(manager): unify logging #22398
fix(manager): unify logging #22398
Conversation
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.
Are the optional packageFile?
params due to tests?
Co-authored-by: Sebastian Poxhofer <secustor@users.noreply.github.com>
Co-authored-by: Sebastian Poxhofer <secustor@users.noreply.github.com>
fixing the optional package file would cause the PR to become at least four times more changes and double of changed files. so I would prefer to do those changes in follow up PR's. also lifting the trace call at beginning of the extract is a good idea for a separate PR. |
🎉 This PR is included in version 35.102.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
Unify parser error logging for allmost all managers.
packageFile
on errorsfileName
topackageFile
Context
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: