Re-order and format disagree on code region formatting

[NicholasBolen]

Format produces:

extends Node


func foo() -> void:
	print("foo")

#region Test Region
func bar() -> void:
	print("bar")
#endregion

Reorder adds an extra new line, producing:

extends Node


func foo() -> void:
	print("foo")


#region Test Region
func bar() -> void:
	print("bar")
#endregion

This code will continue flipping between these states as the code is formatted/reordered. I usually run format then reorder, but now my CI fails since it thinks the code isn't formatted properly.

I prefer the result of reorder as it keeps the consistent 2 blank lines, but either works.

[NathanLovato]

Thanks! Note that when you run reorder, it formats the code, so you don't have to run both. Here, ideally, we should get two lines between functions. Just the formatter is not necessarily aware or checking across region boundaries when a region wraps a function at the moment, but reordering needs its own logic that's very different from the rest of the format and has to do that.

Anyway, long story short, this will be fixed in a future version. But for you now, if you do use the reorder feature, you only need to run that one, and you don't need to run the format command separately.