Skip to content

Conversation

@wbingli
Copy link

@wbingli wbingli commented Aug 3, 2020

Issue #, if available: #282

Description of changes:

As described in #282, the java plugin could be configured somewhere else other than the pom.xml under the resource handler directory.

The tool was trying to help the runtime dependency compatibility check, but it should not have assumption on how to build the package as there are various ways to build the package.

This PR will loose the java plugin version check in pom.xml. If the java plugin dependency doesn't find, don't fail the generate.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@wbingli wbingli changed the title Not fail if java plugin don't in pom.xml Not fail if java plugin doesn't in pom.xml Aug 3, 2020
Copy link
Contributor

@ammokhov ammokhov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@wbingli wbingli merged commit ee42201 into aws-cloudformation:master Aug 18, 2020
@wbingli wbingli deleted the loose_check_pom branch August 18, 2020 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants