Project

General

Profile

Actions

Architecture #8126

closed

Remove trailing space in code file that make merge conflicts

Architecture #8126: Remove trailing space in code file that make merge conflicts

Added by François ARMAND almost 10 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
2
Category:
Architecture - Code maintenance
Target version:
Effort required:
Name check:
Fix check:
Regression:

Description

As we still don't have a linter, we have a lot of files with tailing spaces which cause a huge amount of conflicts in space-unsenstive code file (scala, html, css, js, etc).

We just need to take it down one time for all, and add a linter + git pre-commit hook to take of the solution on the long run.

For now, I'm only taking care of scala project, where there is the more such problem, and the less risk to touch space-sensitive files.

It need to be done in each project, and the merge to hight branch will need to be done with care.


Subtasks 5 (0 open5 closed)

Architecture #8128: Remove trailling whitespace - parent-pomReleasedNicolas CHARLESActions
Architecture #8129: Remove trailling whitespace - rudder-commonsReleasedNicolas CHARLESActions
Architecture #8130: Remove trailling whitespace - scala-ldapReleasedNicolas CHARLESActions
Architecture #8131: Remove trailling whitespace - ldap-inventoryReleasedNicolas CHARLESActions
Architecture #8132: Remove trailling whitespace - cf-clerkReleasedNicolas CHARLESActions

Updated by François ARMAND almost 10 years ago Actions #1

This one is for rudder repos.

Updated by François ARMAND almost 10 years ago Actions #2

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/1072

Updated by François ARMAND almost 10 years ago Actions #3

  • Status changed from Pending technical review to Pending release

Updated by Vincent MEMBRÉ almost 10 years ago Actions #4

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.20, 3.0.15, 3.1.9 and 3.2.2 which were released today.

Actions

Also available in: PDF Atom