[RAT-450] Bug: Deprecation warning does not contain dots at the end of sentences

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.17
• Labels: V0.17
• Assignee: Philipp Ottlinger
• Reporter: Philipp Ottlinger
• Created: 1 week ago 22:28
• Updated: 3 days ago 21:45
• Resolved: 3 days ago 21:45

Description

While enabling the current SNAPSHOT on a project that used v0.16.1 I realized that the format of the error message could be more user-friendly.

1. 1. Shown behaviour

[WARNING] Parameter 'addDefaultLicenseMatchers' (user property 'rat.addDefaultLicenseMatchers') is deprecated: use <config>
[WARNING] Parameter 'excludes' is deprecated: Deprecated for removal since 0.17: Use <inputExclude> instead.
[WARNING] Parameter 'useDefaultExcludes' (user property 'rat.useDefaultExcludes') is deprecated: when set to true specifies that the STANDARD_PATTERNS are excluded, as are the STANDARD_SCMS patterns. Use the various InputExclude and InputInclude elements to explicitly specify what to include or exclude.
[WARNING] Parameter 'useEclipseDefaultExcludes' (user property 'rat.useEclipseDefaultExcludes') is deprecated: when set to true specifies that the ECLIPSE patterns are excluded Use "inputIncludeStd ECLIPSE" to override.
[WARNING] Parameter 'useIdeaDefaultExcludes' (user property 'rat.useIdeaDefaultExcludes') is deprecated: when set to true specifies that the IDEA patterns are excluded Use "inputIncludeStd IDEA" to override.
[WARNING] Parameter 'useMavenDefaultExcludes' (user property 'rat.useMavenDefaultExcludes') is deprecated: when set to true specifies that the MAVEN patterns are excluded Use "inputIncludeStd MAVEN" to override.

1. 1. Expected behaviour

[WARNING] Parameter 'useMavenDefaultExcludes' (user property 'rat.useMavenDefaultExcludes') is deprecated: when set to true specifies that the MAVEN patterns are excluded. Use "inputIncludeStd MAVEN" to override.

This seems to work fine Options as is printed:
[WARNING] Option [-e, --exclude] used. Deprecated for removal since 0.17: Use --input-exclude instead.

WDYT? [e-mail]
 

Issue Links

• links to: GitHub Pull Request #388