Skip to content

Update to Spring Data 2.0.0.RC3 - #12

Merged
reda-alaoui merged 2 commits into
Cosium:masterfrom
marceloverdijk:patch-1
Sep 20, 2017
Merged

reda-alaoui merged 2 commits into
Cosium:masterfrom
marceloverdijk:patch-1

Conversation

@marceloverdijk

Copy link
Copy Markdown
Contributor

This makes it easier for people to clone the repo and build spring-data-jpa-entity-graph:2.0.0-SNAPSHOT locally (as there is no release yet compatible with Spring Boot 2.0)

This makes it easier for people to clone the repo and build spring-data-jpa-entity-graph:2.0.0-SNAPSHOT locally (as there is no release yet compatible with Spring Boot 2.0)
@reda-alaoui

Copy link
Copy Markdown
Member

Hello @marceloverdijk ,

Thanks for the PR.

Travis is complaining about the new dependency.
I guess you didn't have this issue?

@marceloverdijk

Copy link
Copy Markdown
Contributor Author

Hi @reda-alaoui no I did't have this issue, probably because the Spring Data 2.0.0. RC3 artifacts were already in my local Maven repository.

I see Travis is trying to download it from https://repo.spring.io/libs-snapshot-local, and the milestones are stored in https://repo.spring.io/libs-milestone-local/.

I think you have to add that to your src/site/resources/settings.xml configuration and it should work.

@reda-alaoui

Copy link
Copy Markdown
Member

Ok, can you add the milestone repository to src/site/resources/settings.xml to make the build pass?

@marceloverdijk

Copy link
Copy Markdown
Contributor Author

I've added it but still fails.
How does your project resolve the snapshot deps then?

I see this `before_install:

@reda-alaoui

Copy link
Copy Markdown
Member

Correct.
Let's merge it ;)

@reda-alaoui
reda-alaoui merged commit cd119b9 into Cosium:master Sep 20, 2017
@reda-alaoui

Copy link
Copy Markdown
Member

Build OK on master !

@marceloverdijk

Copy link
Copy Markdown
Contributor Author

👍

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.

2 participants