Add support for Spring boot 2.1.0 and Lovelace SR2

[boostchicken]

No description provided.

[boostchicken]

I cant run format on windows its add windows line endings

[codecov-io]

Codecov Report

Merging #213 into master will increase coverage by 0.19%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #213      +/-   ##
===========================================
+ Coverage     65.21%   65.4%   +0.19%     
- Complexity      598     600       +2     
===========================================
  Files            77      77              
  Lines          2038    2038              
  Branches        374     374              
===========================================
+ Hits           1329    1333       +4     
+ Misses          554     550       -4     
  Partials        155     155

Impacted Files	Coverage Δ	Complexity Δ
.../repository/support/DynamoDBRepositoryFactory.java	85.1% <ø> (ø)	20 <0> (ø)	⬇️
.../data/dynamodb/mapping/DynamoDBMappingContext.java	75% <0%> (+25%)	8% <0%> (+2%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f4ba4a...77d9206. Read the comment docs.

[boostchicken]

Loss of code coverage is due to Spring making more interfaces final making Mockito useless.

[boostchicken]

Loss of code coverage issues resolved

[boostchicken]

@derjust any intention of throwing up this snapshot Spring 2.1 is GA and I would love to move away from a local repo

[derjust]

Appreciate it - looks good!

Right now thinking about the various versions that are out there to be supported - honestly dislike that they introduced a new method signature again.

Merging with #17

[boostchicken]

Agree plus all the new final classes, was not hard to get it working, but I shouldn't have had to make any changes at all

[ryonday]

Looks like we independently did much the same work in our respective PRs!

I did manage to get the Mockito stuff working.. Looks like between your PR and mine we can keep the tests and gain Spring Boot 2.1.x compatibility!

Perhaps to make things easier on @derjust we can combine our two PRs

[boostchicken]

I am fine with whatever, I just needed it for my project so I did it, looks like we came to same solution although I think I handled the code coverage a bit different, do a diff and you can see. I think mine is more the "Mockito" way, I could be wrong though. Been a while since I used Mockito.

@ryonday @derjust

[boostchicken]

Also I have some fixed to get it to build on windows environments properly in the POM as well, I know that's rare but it's still useful :)

[boostchicken]

@ryonday let me know if you want to merge into my branch and PR or go the other way around and we'll get the permission sorted and what not.

[boostchicken]

Also @derjust if you need some help administering the project let me know, I've contributed to a few spring data projects in the past (solr, jpa, now dynamo)

[devinsba]

I hope you don't mind @derjust but my team needs this code too so I figured another review might help you finalize this

[boostchicken]

Incorporated @devinsba feedback

[boostchicken]

@derjust just tested this all against SR2 to be sure and we are good.

[derjust]

A thousand thanks to all!
@boostchicken @ryonday @devinsba all appreciate your work!
I merged all of those into branch derjust:Issue_218 to keep all your commits intact & provide credits.

Any help is always highly appreciated and I hope to have more free time soon again.

Still interessted in the changes for a Windows build :-)