Fix apply schemata step doesnt generate formatted code

[hoangatuan]

What it Does

• Closes #263
• Adding a tab into the leading trivia for If body block & Else body block

Note

Since we can't get the Xcode indentation setting directly from Swift, I added a .tabs(1) to the leading trivia for If body block and Else body block by default. As long as the Tab Width and Indent Width settings are equal, the Apply schemata will generate formatted code directly.

In case Tab Width and Indent Width are different, the Apply schemata will not be able to generate formatted code.
However, you can use tools like swiftformat or Command + I to format the code easily

[ZevEisenberg]

Apparently you may be able to use this to call .formatted() at the end to get what you want? I haven't tried it, but it sounds interesting!

[hoangatuan]

Apparently you may be able to use this to call .formatted() at the end to get what you want? I haven't tried it, but it sounds interesting!

@ZevEisenberg Thanks for your suggestion. My current approach still faces some limitations here.
Let me try with your suggestion to see if it works

[hoangatuan]

Hi @ZevEisenberg, I pushed the latest change. I've updated the logic to:

• Follow apple-swift-format approach to allow the developer to configure indentation in the configuration file.
• I realized that appending only 1 tab to the leading trivia of the syntax is not enough as discuss here. I follow realm-swift-format approach to solve the problem

[rakaramos]

hey @hoangatuan thanks for effort. I'm wondering why not just use this and skip the property on the configuration. If we that the result is much better because we are using something that goes beyond the indentation we can leave the tab vc spaces war for the rest of the people 😆

[hoangatuan]

@rakaramos Oh wow I don't know about that  😂 That's work too But that code uses indentation = .spaces(2) by default, so should we still support users to configure the indentation in the yaml configure file? (Similar to what apple-swift-format has done)

[rakaramos]

@hoangatuan I think we can live we 2 spaces 😄 it's not super important for the users to lint something as ephemeral as the mutated source code. this is why we disable all the linters.

[hoangatuan]

@rakaramos I've updated the PR base on your suggestion