andrey-petrenko-db commented on a change in pull request #610:
URL: https://github.com/apache/maven/pull/610#discussion_r748749441
##########
File path:
maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java
##########
@@ -140,21 +145,22 @@ else if ( Artifact.SCOPE_TEST.equals( classpath ) )
return Collections.unmodifiableCollection( scopes );
}
- public void execute( MavenSession session, List<MojoExecution>
mojoExecutions, ProjectIndex projectIndex )
- throws LifecycleExecutionException
+ public void execute( final MavenSession session,
+ List<MojoExecution> mojoExecutions,
+ final ProjectIndex projectIndex ) throws
LifecycleExecutionException
{
- DependencyContext dependencyContext = newDependencyContext( session,
mojoExecutions );
+ final DependencyContext dependencyContext = newDependencyContext(
session, mojoExecutions );
- PhaseRecorder phaseRecorder = new PhaseRecorder(
session.getCurrentProject() );
+ final PhaseRecorder phaseRecorder = new PhaseRecorder(
session.getCurrentProject() );
Review comment:
since java8 no need to make them final
--
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: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
