rudder icon indicating copy to clipboard operation
rudder copied to clipboard
verified publisher icon Normation

Fixes #21741: Auto-format scala code

Open fanf opened this issue 3 years ago • 7 comments

https://issues.rudder.io/issues/21741

The formating is done with the process explained in ticket comment, everything is repeatable now. In other branches, some more relative imports may exists, but it should be enought to add them in the sed list.

Be careful to always do the last two mvn command: it happens that something breaks in very funny and novel ways, like something really looking like a comment but no, it's real scala code.

fanf avatar Sep 08 '22 16:09 fanf

PR updated with a new commit

fanf avatar Sep 08 '22 16:09 fanf

PR updated with a new commit

fanf avatar Sep 08 '22 17:09 fanf

PR updated with a new commit

fanf avatar Sep 09 '22 08:09 fanf

PR updated with a new commit

fanf avatar Sep 09 '22 10:09 fanf

PR updated with a new commit

fanf avatar Sep 09 '22 10:09 fanf

PR updated with a new commit

fanf avatar Sep 09 '22 15:09 fanf

PR updated with a new commit

fanf avatar Sep 09 '22 17:09 fanf

PR rebased

fanf avatar Oct 20 '22 14:10 fanf

PR rebased

fanf avatar Oct 20 '22 17:10 fanf

This PR is not mergeable to upper versions. Since it is "Ready for merge" you must merge it by yourself using the following command: rudder-dev merge https://github.com/Normation/rudder/pull/4473 -- Your faithful QA Kant merge: "It is beyond a doubt that all our knowledge begins with experience." (https://ci.normation.com/jenkins/job/merge-accepted-pr/61170/console)

Normation-Quality-Assistant avatar Oct 20 '22 18:10 Normation-Quality-Assistant

OK, merging this PR

fanf avatar Oct 20 '22 18:10 fanf