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.

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

feat: add astx transformer #683

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jedwards1211
Copy link

@jedwards1211 jedwards1211 commented Dec 7, 2022

This is self-promotion, since I'm the author of astx. It's a sweet JS/TS transformer for doing pattern-based structural search and replace.

Note: I made transformerMiddleware pass the parser and parserSettings to the transform function...the way it was being done didn't make a whole lot of sense, for instance with jscodeshift it templates the selected parser into the transform code at first, but you can still select a different parser afterwards in astexplorer, making the two out of sync. To me it makes more sense for the transformer to always use the parser and settings selected in the workbench (if applicable?)

Thanks so much for all the work you've done on astexplorer! Without it I may never have gotten into writing codemods.

@jedwards1211
Copy link
Author

Oh it doesn't automatically rerun the transformer when I select a different parser in the workbench. Need to make it do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant