Skip to content

Conversation

@wilkinsona
Copy link
Member

Configure the Spring IO plugin such that it's only applied when the build is run with -PplatformVersion=<version>. This platformVersion property is used to determine the version of the Platform that will
be used when running the springIoCheck task. The plugin can be used by running a build as follows:

./gradlew clean springIoCheck -PplatformVersion=1.0.0.BUILD-SNAPSHOT -PJDK8_HOME=…

This will test the project on JDK 8 using the dependencies defined in the latest snapshot of Spring IO Platform 1.0.0.

build.gradle Outdated
Copy link
Member

Choose a reason for hiding this comment

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

???

Copy link
Member Author

Choose a reason for hiding this comment

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

Sorry, that was a leftover from my testing against a local build of the platform. I'll update the PR.

@artembilan
Copy link
Member

I see Spring WS has custom test task, too, as it is with SI.
So, why don't you take care about

tasks.withType(Test).all

?
The same for AMQP

@wilkinsona
Copy link
Member Author

Thanks for spotting that, Artem. Unlike the problems with had with SI, Spring WS builds cleanly without making that change. That said, it's probably a good idea to apply the same configuration.

Configure the Spring IO plugin such that it's only applied when the
build is run with -PplatformVersion=<version>. This platformVersion
property is used to determine the version of the Platform that will
be used when running the springIoCheck task. The plugin can be used
by running a build as follows:

./gradlew clean springIoCheck -PplatformVersion=1.0.0.BUILD-SNAPSHOT -PJDK8_HOME=…

This will test the project on JDK 8 using the dependencies defined in
the latest snapshot of Spring IO Platform 1.0.0.
@artembilan
Copy link
Member

I can't say LGTM, because I don't have WS sources to check it localy.
However for GH it's OK to me

poutsma pushed a commit that referenced this pull request May 14, 2014
@poutsma poutsma merged commit c36b1dd into spring-projects:master May 14, 2014
@poutsma
Copy link
Contributor

poutsma commented May 14, 2014

Merged, thanks!

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