Skip to content
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.

Sign up for GitHub

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

Jump to bottom

🗺 Remove jscodeshift #5140

Closed
1 task done
mjackson opened this issue Jan 19, 2023 · 0 comments
Closed
1 task done

🗺 Remove jscodeshift #5140

mjackson opened this issue Jan 19, 2023 · 0 comments
Assignees
@pcattori

Comments

@mjackson
Copy link
Member

mjackson commented Jan 19, 2023 •
edited by pcattori
Loading

The only reason we still have it is for TS -> JS conversion in the create-remix CLI. jscodeshift is slow and can cause tests to timeout. its a big dependency too. it outputs a bunch of noise whenever tests are run. it produces a bunch of deprecated warnings when a new Remix app is created. and it makes the CLI slower.

PRs
@mjackson mjackson added bug:unverified bug Something isn't working and removed bug:unverified labels Jan 19, 2023
@MichaelDeBoey MichaelDeBoey added 🗺 Roadmap and removed bug Something isn't working labels Jan 19, 2023
@MichaelDeBoey MichaelDeBoey changed the title Remove jscodeshift 🗺 Remove jscodeshift Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcattori pcattori

Labels
None yet
Projects
Roadmap
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mjackson @pcattori @MichaelDeBoey