Bug: "Preserve existing aliases section style" should check Linter's array section style setting to avoid >1 lint about obsidian-linter HOT 6 CLOSED

Hey @redactedscribe . I am not sure I 100% follow the issue here. I can see that it is not formatting things as a multiline array on the first lint, but that seems to make sense to me personally. The reason why things look odd is because of how properties works (see #876). Essentially, an empty mutli-line array according to properties is []. In order to accommodate that, the Linter has to assume that an empty multi-line array is []. This means that when you get to YAML Title alias you are working with a single-line array. Thus when the value is inserted you get a single line array since you have the option to keep the current key type.

I am thinking I can do a check that says if the original array was [] go with whatever the default type is.

from obsidian-linter.

I think the fix for this should not be too hard. I am hoping it will be a simple fix.

from obsidian-linter.

Looks like the fix was pretty simple. It should be on master soon.

from obsidian-linter.

I have a PR ready that will hopefully handle this. I am just waiting on the UTs to run.

from obsidian-linter.

The fix should now be on master and it should go out in the next release. Please let us know if there is an issue with either.

Also, thanks for pointing out this inconsistency. I did not realize it had been missed when the changes were made for #876 .

from obsidian-linter.

Thanks as always @pjkaufman. I'll let you know if the fix works on my end.

(Also, I may have been consfusing as I said "produce the second codeblock above" when I really meant "the codeblock above" (i.e. the third). Hopefully you saw through that typo.)

from obsidian-linter.