Architecture #3256
closed
Remove trailling space at the end of lines, and line full of whitespace character
Description
In order to have a nice and clean code, I suggest to remove all those whitespace characters that should not be here.
The only files targeted in that user story are the Scala files (because they deserve it).
This can be done using a smart regexp.
Updated by Vincent MEMBRÉ about 11 years ago
- Status changed from New to 13
Here are the pull requests :
Commons : https://github.com/Normation/rudder-commons/pull/6
Scala-LDAP : https://github.com/Normation/scala-ldap/pull/5
Cf-clerk : https://github.com/Normation/cf-clerk/pull/6
Inventory : https://github.com/Normation/ldap-inventory/pull/14
Rudder : https://github.com/Normation/rudder/pull/118
Updated by Vincent MEMBRÉ about 11 years ago
- Subject changed from [Breaking change] Remove trailling space at the end of lines, and line full of whitespace character to Remove trailling space at the end of lines, and line full of whitespace character
Updated by François ARMAND about 11 years ago
I believe that we need to set-up a Scala style checker1 to track that kind of stuff automatically. And configure our dev environment to automatically do that kind of stuff each time we save a file
(in Eclipse, see Preference -> Java -> Editor -> Save actions)
Vincent, could you open a ticket for that is none already exists?
Thanks,
Updated by Vincent MEMBRÉ about 11 years ago
- Status changed from 13 to Pending release
- % Done changed from 0 to 100
Applied in changeset 1d91ae236d7788cd284cab9158f8aaa6cce75d53.
Updated by Anonymous about 11 years ago
Applied in changeset 420fae816072cf888c34a2ed166766116303fb5b.
Updated by Jonathan CLARKE about 11 years ago
- Tracker changed from User story to Architecture
This is clearly not a user story, it is a long way from what users should see. Please use the "Architecture" tracker for non-user-visible changes.
Updated by Jonathan CLARKE about 11 years ago
- Status changed from Pending release to Released
This ticket has been addressed in version 2.6.0~beta1 of Rudder, which has just been released. Please see the changelog here: https://www.rudder-project.org/foswiki/System/Documentation:ChangeLog26.