-
Notifications
You must be signed in to change notification settings - Fork 491
Closed
Description
Describe the bug
The temp library was replaced by tmp in #633, and published in https://github.com/facebook/jscodeshift/releases/tag/v17.1.0.
However, the published artifact for v17.1.0 still calls temp, and it breaks in consumers.
Steps to reproduce
$ test-jscodeshift> npm install [email protected] --save-exact
$ test-jscodeshift> npx jscodeshift --version
jscodeshift: 17.1.0
- babel: 7.26.0
- babylon: 7.26.2
- flow: 0.251.1
- recast: 0.23.9
$ test-jscodeshift> grep -rnI 'temp' node_modules/jscodeshift/dist/Runner.js
node_modules/jscodeshift/dist/Runner.js:17:const temp = require('temp');
node_modules/jscodeshift/dist/Runner.js:193: temp.open({ prefix: 'jscodeshift', suffix: ext }, (err, info) => {
node_modules/jscodeshift/dist/Runner.js:314: temp.cleanupSync();Observed behavior
The temp is still imported in published artifacts
Expected behavior
The tmp should be imported in published artifacts instead
Additional context
Bug report in consumer aws/aws-sdk-js-codemod#956
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels