Re: [papyrus-rt-dev] Code formatting and checkstyle rules

[Céline JANSSENS <celine.janssens@xxxxxxxxxxx>]

Hello Philip,

I completely agree with you, it must be done soon, as the project is not so big yet.

For your information, I'm working on this already. I  modified the pom file on the Papyrus RT project to manage a Sonar Job, which is running every day but still have some dependancies issues due to profile setting.
I asked Benoit to have a look, but he is quite busy until the end of the week.

In addition, Papyrus already has a formatter, a cleanup and a Code Template file.
Which is the one I use in Papyrus RT. You can find those templates here:



All4Tec is used to work with checkstyle. Which is quite efficient. I will adapt the one we use for Papyrus RT and once in place we could discuss about its content. And disable or enable rules accordingly.

The Checkstyle file is based on the Sonar standard rules.

A discussion has been started about the Development best practices here:
https://wiki.eclipse.org/Papyrus/Neon_Work_Description/Discussions/Development_rules

It could be a good starting point to our checkstyle.

Best regards
Céline

Le 16/03/2016 14:01, Philip Langer a écrit :

Hi Papyrus-RT-dev team,

we noticed that there don't seem to exist any central code formatting settings and checkstyle rules. Before the project gets too large, it may pay off to introduce and consistently use such settings and rules. Otherwise it may soon get too cumbersome to introduce them once the code base gets too large.

We think that they'd help to keep the project maintainable on the long run and avoid tedious git conflicts due to inconsistent formatting, etc.

What do you think? We'd volunteer to propose them in a new gerrit patch.

Thanks and best wishes,

Philip

--

Dr. Philip Langer

Senior Software Architect / General Manager
EclipseSource Services GmbH

_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

--

	Céline JANSSENS Software Engineer +33 (0)2 44 47 23 23
	Mail : cej@xxxxxxxxxxx
6 rue Léonard De Vinci - BP 0119 - 53001 LAVAL Cedex - FRANCE www.all4tec.net