All,

We probably should have adhered to this long ago, but due to years of
development we now have a diverging code base regarding code
formatting.
While this is not a cardinal sin, it is rather easy to fix, and to
prevent e.g. windows line endings to rewrite the whole file.

At dayjob we use the spotless maven plugin to reformat and check the
formatting of our code bases. This works wonderfully, and allows
agents to also write consistent code.

I've asked my agent to distill the "Wicket Code Formatting Rules" from
the source code and make it a spotless profile. I also asked it to
export it into eclipse and intellij formatter profiles.

This is marked under this ticket:

I have prepared 3 PRs:
- master
- wicket-11.x
- wicket 10.x

This will require existing PRs to be reformatted prior to merge, but I
don't think that is a big issue.

WDYT? Should we adopt this:

- reformat all active code bases
- implement a CI gate for code formatting issues

Martijn

Reply via email to