Giovds commented on a change in pull request #678:
URL: https://github.com/apache/maven/pull/678#discussion_r809466481
##########
File path:
maven-model-builder/src/main/java/org/apache/maven/model/interpolation/ProblemDetectingValueSource.java
##########
@@ -60,12 +68,12 @@ public Object getValue( String expression )
if ( value != null && expression.startsWith( bannedPrefix ) )
{
- String msg = "The expression ${" + expression + "} is deprecated.";
+ String msg = "The expression ${" + expression + "} is " + status +
".";
Review comment:
These changes were made for the initial idea of making the build fail on
`pom.` prefixes. Since we are not failing the build anymore these changes have
become unnecessary. Once support for prefixless expressions has been removed
this class can be removed entirely. I've reverted the changes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]