Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tycho-dev] Formater profile used in Tycho
FWIW last year CDT reformatted all of its code consistently, adding the profile and all settings into the projects. As part of the gerrit check process we make sure that all code is formatted to project specifications, including some other rules such as no trailing whitespace.

Have a look at gerrit CI job and the script that does the checking.

Jonah

~~~
Jonah Graham
Kichwa Coders
www.kichwacoders.com


On Tue, 29 Oct 2019 at 08:44, Aleksandar Kurtakov <akurtako@xxxxxxxxxx> wrote:


On Tue, Oct 29, 2019 at 12:44 PM Mickael Istria <mistria@xxxxxxxxxx> wrote:


On Tue, Oct 29, 2019 at 11:29 AM Christoph Läubrich <laeubi@xxxxxxxxxxxxxx> wrote:
- is there any preffered profile and where do I find it?

No, there is no profile.

- would it be suitable to include that in the tycho-setup directory?

Yes, but what's even more important if we want to enforce/encourage a formatting is that the .settings/ of all modules do reference the same formatter and enable some format on save actions.
Actually, we could stick to the default Eclipse formatter that's built-in (so we wouldn't need to share it) but just ensure we enable this formatter in .settings/

+1000 here. Sticking to built-in formatter is the best option.
 
 
- should I always format the whole file or only edited lines?

I don't think it's important enough for project perspective to make chose one or the other. In general, it's up to the developer to do what's best for them, they're trusted to do the right thing and the project doesn't force a workflow or another.

--
Mickael Istria
Eclipse IDE developer, for Red Hat Developers
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://www.eclipse.org/mailman/listinfo/tycho-dev


--
Alexander Kurtakov
Red Hat Eclipse Team
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://www.eclipse.org/mailman/listinfo/tycho-dev

Back to the top