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

Update prettier-plugin-organize-imports #58

Merged
merged 1 commit into from
Nov 4, 2022

Conversation

quesabe
Copy link
Collaborator

@quesabe quesabe commented Nov 4, 2022

The plugin is just a wrapper around TypeScript organizeImports function. The previous version of the plugin does not work with TypeScript 4.8, because of changed way of handling non-existing methods on a host object - the recent TypeScript implementations throws an error instead of silently continuing. The issue was resolved with the plugin providing an extended host object implementation with all necessary methods present.

Closes PLA-142.

The plugin is just a wrapper around TypeScript organizeImports function. The previous version of the plugin does not work with TypeScript 4.8, because of changed way of handling non-existing methods on a host object - the recent TypeScript implementations throws an error instead of silently continuing. The issue was resolved with the plugin providing an extended host object implementation with all necessary methods present.
@quesabe quesabe requested a review from gvidon November 4, 2022 07:54
@quesabe quesabe self-assigned this Nov 4, 2022
@linear
Copy link

linear bot commented Nov 4, 2022

PLA-142

@gvidon gvidon merged commit cbc5171 into master Nov 4, 2022
@gvidon gvidon deleted the update-prettier-plugin-organize-imports branch November 4, 2022 14:59
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.

2 participants