这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@DanielFran
Copy link
Member

@DanielFran DanielFran commented Nov 30, 2017

See http://mapstruct.org/news/2017-10-18-mapstruct-1_2_0_Final-is-out/

  • Please make sure the below checklist is followed for Pull Requests.

  • Travis tests are green

  • Tests are added where necessary

  • Documentation is added/updated where necessary

  • Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

@DanielFran
Copy link
Member Author

Align versions with jhipster-dependencies 0.1.0: upgrade spring-boot to 1.5.9 jhipster/jhipster-dependencies#23

@erikkemperman
Copy link
Member

erikkemperman commented Dec 1, 2017

FYI we’ll also need some changes to adapt to new Hazelcast when jhipster-dependencies is released with the new spring-platform and spring-boot. I have those figured out already.

@erikkemperman
Copy link
Member

@DanielFran Maybe you should change the title of this PR, it's more about upgrading spring-platform / spring-boot right? Of course this PR has to wait until a new release of jhipster-dependencies is out with those changes in, and then it will be 0.1.1 or 0.2.0.

@DanielFran DanielFran changed the title upgrade mapstruct to 1.2.0.Final Upgrade to spring-boot 1.5.9 and mapstruct to 1.2.0.Final Dec 1, 2017
@deepu105 deepu105 closed this Dec 2, 2017
@deepu105 deepu105 reopened this Dec 2, 2017
Copy link
Member

@erikkemperman erikkemperman left a comment

Choose a reason for hiding this comment

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

This looks solid, although I would have probably separated the spring-boot and mapstruct upgrades in two PRs.

But this can't be merged (nor tested) until this PR in jhipster-dependencies is merged and released:
jhipster/jhipster-dependencies#23
And at that point the version of jhipster-dependencies will have to be incremented in this branch here.

@DanielFran
Copy link
Member Author

DanielFran commented Dec 4, 2017

In fact, this was my intention to have 2 different PR but since I am a noob with git (it is my 2nd time I use it, both with jhipster projects), this has been merged in 1. This is why initially the name was only about mapstruct...

@pascalgrimaud
Copy link
Contributor

@DanielFran : no worry, don't hesitate to ask advice, reviews etc.

Thanks and congrats for your 1st contribution !

@erikkemperman
Copy link
Member

All right, it looks like jhipster-dependencies is now released at 0.1.4 with the platform upgrade in it:
jhipster/jhipster-dependencies@c994969

But to work in the generator, we'll also need a patch to accommodate the new Hazelcast version. If you want to copy/paste, please feel free, I've got the needed changes here:
erikkemperman@673b5c5

@jdubois
Copy link
Member

jdubois commented Dec 5, 2017

Thanks @erikkemperman - I didn't see your comment yesterday, but that's all merged now, so it should be good

@jdubois jdubois merged commit e7825a1 into jhipster:master Dec 5, 2017
@jdubois jdubois added this to the 4.12.0 milestone Dec 8, 2017
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.

5 participants