-
Notifications
You must be signed in to change notification settings - Fork 39
tabs changed to spaces #464
Comments
i sense that preserving formats is going to be an never ending game of whack-a-mole 😅 i do want to point out that this is only the case if the user doesn't have |
I think in the past I made sure that this worked as expected. We might need to try to remove this method for a second to see if this might already fix this. svelte-add/packages/ast-tooling/index.ts Line 81 in 2f9a963
|
I haven't looked at the code that closely, but was hoping if we got the functionality to live in the core functions that it would handle it for most cases
Yeah, I think that's the case with most of the stuff found in the past day or so. Understandable since there's been a lot of refactoring. I think with some tests hopefully we can avoid having to deal with too many regressions (#461) |
You're correct, it is the removal of the locations causing this! Rather than removing |
Topic
mdsvex
Version
2.3.2
Expected behavior
leave the tabs in place
it looks like there's a utility function we can use for this now: #463 (comment)
Actual behavior
the mdsvex adder changes all of the tabs to spaces in
svelte.config.js
The text was updated successfully, but these errors were encountered: