Project

General

Profile

Architecture #11968

Architecture #11955: Remove most scala compilation warning

Remove "N-uple adaptation" warning

Added by François ARMAND about 2 years ago. Updated about 2 years ago.

Status:
Released
Priority:
N/A
Category:
Architecture - Refactoring
Target version:
Effort required:

Description

Scala is deprecating the use of adaptation from n-args to tuple, so for example you need to use:

val x: Option[(Int, Int)] = Some((1,2))

This is to remove some ambiguities that can lead to non-expected result in some case.

I'm not sure I really like the result. In some caes, the alternative tuple notation works very well, but not always:

... myLiftXmlElement % (("class", "error"))  // bof

... myLiftXmlElement % ("class" -> "error")) // nice!

But in all case, we should have the smallest number of warnings (ideally, 0) with -Xlint option enabled.


Subtasks

Architecture #11975: Remove "N-uple adaptation" warning (missing one)ReleasedVincent MEMBRÉActions
#1

Updated by François ARMAND about 2 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/1828
#2

Updated by Rudder Quality Assistant about 2 years ago

  • Assignee changed from Vincent MEMBRÉ to François ARMAND
#3

Updated by François ARMAND about 2 years ago

  • Status changed from Pending technical review to Pending release
#4

Updated by Vincent MEMBRÉ about 2 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.3.0~beta1 which was released today.

Also available in: Atom PDF