Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Fix Maven build-time warnings #29

Merged
merged 2 commits into from
Feb 4, 2016
Merged

Fix Maven build-time warnings #29

merged 2 commits into from
Feb 4, 2016

Conversation

olivierdagenais
Copy link
Contributor

There were a couple of warnings during a Maven build that I took care of; see the individual commits for details.

Manual testing

Set my JAVA_HOME to JDK 1.6.0_20 on Windows and ran mvn clean verify. There are no longer any warnings.

Mission accomplished!

Oli Dagenais added 2 commits February 4, 2016 10:59
This removes the following warning when building or
running tests with JDK 6:

Feb 4, 2016 10:56:26 AM
org.codehaus.groovy.runtime.m12n.MetaInfExtensionModule newModule
WARNING: Module [groovy-all] - Unable to load extension class
[org.codehaus.groovy.runtime.NioGroovyMethods]
This eliminates the following warnings during build:

[WARNING] The expression ${artifactId} is deprecated. Please use
${project.artifactId} instead.
[WARNING] The expression ${version} is deprecated. Please use
${project.version} instead.
@yacaovsnc
Copy link
Member

LGTM!

yacaovsnc added a commit that referenced this pull request Feb 4, 2016
Fix Maven build-time warnings
@yacaovsnc yacaovsnc merged commit 4673a81 into master Feb 4, 2016
@olivierdagenais olivierdagenais deleted the fix_warnings branch February 4, 2016 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants