bikehopper / bikehopper-ui

Friendly bike+transit directions (frontend)
Other
27 stars 1 forks source link

Remove vite-plugin-checker #356

Closed Andykmcc closed 5 months ago

Andykmcc commented 5 months ago

The TSC output was overwriting the vite output in the terminal. its not clear to me what this checker even did since everything worked without it.

https://github.com/bikehopper/bikehopper-ui/issues/347

Andykmcc commented 5 months ago

Is the goal to have TS and ESlint warnings/errors/whatever output to the terminal so everyone can see them regardless of editor/IDE?

graue commented 5 months ago

Yeah, that's the goal. Can you see if, rather than removing the vite checker, it can be used to show both TS and lint status?

Andykmcc commented 5 months ago

i dont think we should use the checker plugin. i mentioned this in discord, but i don't think its really maintained. it has lots of outstanding issues and PR. There is an open issue from last October for this exact bug.