[MNG-7754] Improvement and extension of plugin validation

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.9.1
• Fix Version/s: 3.9.2, 4.0.0-alpha-7, 4.0.0
• Component/s: Core
• Assignee: Tamas Cservenak
• Reporter: Tamas Cservenak
• Created: 31/Mar/23 17:43
• Updated: 11/May/23 13:22
• Resolved: 12/Apr/23 10:21

Description

Some users when see following warning:

[INFO] --- remote-resources:1.7.0:process (process-resource-bundles) @ maven ---
[WARNING] Parameter 'localRepository' is deprecated core expression; Avoid use of ArtifactRepository type. If you need access to local repository, switch to '${repositorySystemSession}' expression and get LRM from it instead. 

on their console, immediately grep their $HOME to find out that they DO HAVE afore mentioned string present in their settings.xml, and then scratch their head how to get rid of it.

Hence, we should improve error message – at least add some clue that message targets given Mojo developers (as message appears immediately under Mojo execution log message) and not users. Best users could do is nag Mojo developers, and not us, to make message disappear.

Improvements:

• report at end, instead multiple times same warning (for reactor builds)
• possibility to suppress validation
• new check: maven-compat, maven2, p-c-d, mixed maven verions,, wrong scopes (similar as m-p-p does on build time)

Issue Links

• fixes: MNG-7712 Core should issue a warning if plugin depends on maven-compat
• is related to: MNG-7705 Sporadic failures on multiple builds sharing the same local repo when writing the .lastUpdated file
• relates to: MNG-7767 Tone down plugin validation report
• links to: GitHub Pull Request #256
• links to: GitHub Pull Request #1079