Skip to content

Commit

Permalink
fix standalone transform runs - resolve path of transform
Browse files Browse the repository at this point in the history
in the issue you linked [1] from jscodeshift, i think you forgot the `path.resolve` for non-http(s) imports (not even sure how they'd work tbh).

i tried running a local transform and it didn't work, but with this fix it did regardless of where i called the `packages/cli/bin/codeshift-cli.js` from. i think you might've been using the packages and not the --transforms flag so i understand why you could've missed it 😅

tbh there are more improvements to make w/ the local transforms (see [2]) but this is the first step :D

[1] facebook/jscodeshift#398
[2] #48 (comment)

Signed-off-by: Kipras Melnikovas <[email protected]>
  • Loading branch information
kiprasmel committed Jul 8, 2022
1 parent ed819f7 commit 999edd6
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions packages/cli/src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -231,9 +231,10 @@ export default async function main(paths: string[], flags: Flags) {
);

for (const transform of transforms) {
console.log(chalk.green('Running transform:'), transform);
const resolvedTransformPath = path.resolve(transform);
console.log(chalk.green('Running transform:'), resolvedTransformPath);

await jscodeshift.run(transform, paths, {
await jscodeshift.run(resolvedTransformPath, paths, {
verbose: flags.verbose,
dry: flags.dry,
print: true,
Expand Down

0 comments on commit 999edd6

Please sign in to comment.