feat/migrate-antd-removed-component-migration #517
Conversation
Current dependencies on/for this PR:
This stack of pull requests is managed by Graphite. |
@@ -0,0 +1,27 @@ | |||
{ | |||
"name": "@codemod-registry/antd-5-v5-remove-style-import", |
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.
this name is odd, how did it happen?
codemods/antd/5/utils/marker.js
Outdated
); | ||
} | ||
|
||
async function getDependencies() { |
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.
is this used anywhere? I think it will crash as direct FS operations are unsupported in our model of execution.
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.
only markDependency is used, but its also using fs
. Should we reimplement this codemod as Filemod?
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.
I would remove the parts that are not used and comment out the usage of mark dependency. We may consider moving to a filemod later on.
c116fe8
to
a71ba5f
Compare
7cb8989
to
98c79fd
Compare
98c79fd
to
7519961
Compare
feat/migrate-antd-removed-component-migration
feat/migrate-antd-removed-component-migration