Project

General

Profile

Actions

Architecture #12558

closed

Enforce stricter checks by scala compiler with Xlint flags

Added by François ARMAND over 6 years ago. Updated over 6 years ago.

Status:
Released
Priority:
N/A
Category:
Architecture - Internal libs
Target version:
Effort required:
Name check:
Fix check:
Regression:

Description

We don't use scala compiler to its full power. In fact, we don't even use it for simple things like unused imports.

In #11955, we did an ad hoc compilation with the interesting flags, but it is not persistent and we need to enforce them in Maven pom to be sure to not drift from the clean state.

https://tpolecat.github.io/2017/04/25/scalac-flags.html gives an overview of all the interesting flags. We won't use them all right now, because our code base it not ready :)

We will also need to be able to selectively silent some warning, in particular in the case of deprecationWarning for things that can't be replaced (easely or at all, like Thread.stop). For that, we will use the silencer plugin: https://github.com/ghik/silencer

This change will span modification on all scala projects, this ticket handle modifications in `parent-pom` one.


Subtasks 5 (0 open5 closed)

Architecture #12559: Correct scalac warnings in rudder-commonsReleasedVincent MEMBRÉActions
Architecture #12560: Correct scalac warnings in scala-ldapReleasedVincent MEMBRÉActions
Architecture #12561: Correct scalac warnings in ldap-inventoryReleasedVincent MEMBRÉActions
Architecture #12562: Correct scalac warnings in rudderReleasedVincent MEMBRÉActions
Bug #12644: Stacktrace when displaying comet componentsReleasedFrançois ARMANDActions

Related issues 1 (0 open1 closed)

Has duplicate Rudder - Architecture #2619: Add a style checking tool for ScalaRejectedActions
Actions #1

Updated by François ARMAND over 6 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-parent-pom/pull/53
Actions #2

Updated by François ARMAND over 6 years ago

We also update some libraries that help with false positives with these compiler flags.

Actions #3

Updated by François ARMAND over 6 years ago

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

Updated by Nicolas CHARLES over 6 years ago

Actions #5

Updated by Vincent MEMBRÉ over 6 years ago

  • Category changed from Web - Maintenance to Architecture - Internal libs
Actions #6

Updated by Vincent MEMBRÉ over 6 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.3.2 which was released today.

Actions

Also available in: Atom PDF