Skip to content

Auto import order suggestions #32

Draft
peppineddu5 wants to merge 4 commits into
mainfrom
auto-import-order-suggestions
Draft

Auto import order suggestions #32
peppineddu5 wants to merge 4 commits into
mainfrom
auto-import-order-suggestions

Conversation

@peppineddu5

Copy link
Copy Markdown
Contributor

what

Order of imports is: "./", "../", "@/", "effect-app"

Don't forget to set the version of typescript on Workspace.
image

@patroza

patroza commented Dec 12, 2024

Copy link
Copy Markdown
Member

pattern is wrong

@patroza

patroza commented Jan 9, 2025

Copy link
Copy Markdown
Member

@peppineddu5 I think we should add "effect-app" too, before effect-app/*

regardless doesnt seem to work in frontend vue files
Screenshot 2025-01-09 at 11 19 08
Screenshot 2025-01-09 at 11 25 42

Also isn't preferring the local resources/lib over effect-app it seems:
Screenshot 2025-01-09 at 11 27 18

@peppineddu5

Copy link
Copy Markdown
Contributor Author

I'm also trying to figure out the problem because the plugin is starting up successfully from the ts-server logs, but when I try to add a log inside the function that reorders the imports, it doesn't start. I've also checked the moveUpPatterns and changed the version of typescript but nothing.
I'll try to see if I can figure out the problem, otherwise I'll send an issue to the pr

@patroza patroza marked this pull request as draft January 10, 2025 11:21
@patroza

patroza commented Jan 10, 2025

Copy link
Copy Markdown
Member

@peppineddu5 thanks!

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