Álvaro Queiroz
Álvaro Queiroz
hello @mgyucht do you think this is ready for review?
hello @alexott, this change would not break line1\nline2" -> "line1line2 I have even a test for that this will remove both, empty lines and empty lines with spaces, both would...
I will add the integration test later today @mgyucht
@mgyucht I have added one acceptance test. I have added two steps, one creates the view and the second just adds spaces and empty lines to the view definition. It...
hello @mgyucht do you think it is worth proceeding with the review of this PR or I should close it? I just wanted to fix this bug, but if the...
done @nkvuong let me know if you need anything else
@nkvuong I have synced my fork with the master, the conflict is still present?
@nkvuong fixed 
@nkvuong I think I'm not setting up the test correctly for some reason. It seems to me that the Update flag will trigger an update regardless of the supressdiff method....
just fyi @nkvuong the test is passing because of [this line ](https://github.com/databricks/terraform-provider-databricks/pull/3361/files#diff-72a91d237a7ff752570e5ff10f2245c6f66e0014ace5439104cdcb94f40cb9a4R536) I will get the test with ExpectedDiff in place