-
-
Notifications
You must be signed in to change notification settings - Fork 20
feat(logger): add codemode name #101
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
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.
Pull Request Overview
This PR introduces a configurable codemod name for log output, enabling dynamic updates to logger prefixes.
- Adds a new exported function setCodemodName to update the codemod name.
- Updates log statements and test snapshots to reflect the codemod name change.
- Updates the workflow file to set the codemod name as needed.
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
File | Description |
---|---|
utils/src/logger.ts | Added codemodName variable and exported setCodemodName function. |
utils/src/logger.test.ts | Updated tests to use setCodemodName and validate log output changes. |
utils/src/logger.test.snap.cjs | Updated snapshot values to reflect the new codemod name usage. |
recipes/correct-ts-specifiers/src/workflow.ts | Imported and applied setCodemodName for setting the codemod name. |
Comments suppressed due to low confidence (1)
utils/src/logger.ts:8
- The default codemod name 'nodjs-codemod' appears to have a potential typo. If it is meant to be 'nodejs-codemod', consider updating the default value for clarity.
let codemodName = 'nodjs-codemod';
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.
Multiple codemods could be running at the same time as part of a multi-part migration. In that case, the module-level "name" will get clobbered.
🤦🏼 you have right |
BTW |
09d7f26
to
b4ff835
Compare
No description provided.