lloydmeta / enumeratum

A type-safe, reflection-free, powerful enumeration implementation for Scala with exhaustive pattern match warnings and helpful integrations.
MIT License
1.18k stars 146 forks source link

Upgrade Play & Play JSON to 3.0 #381

Closed gaeljw closed 7 months ago

gaeljw commented 8 months ago

Supersedes https://github.com/lloydmeta/enumeratum/pull/380

As discussed in https://github.com/lloydmeta/enumeratum/pull/380, Play 2.x support is discontinued in favour of Play 3.x unless there is a strong demand for it.

Closes https://github.com/lloydmeta/enumeratum/issues/382

gaeljw commented 8 months ago

I don't understand why 2.12 build fails but for sure it's related to my attempt to not build enumeratum-play for 2.12 anymore. Will investigate a bit more tomorrow. Unless you've got any idea.

codecov-commenter commented 8 months ago

Codecov Report

Merging #381 (6b0828e) into master (dba1630) will increase coverage by 0.19%. The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master     #381      +/-   ##
==========================================
+ Coverage   85.52%   85.71%   +0.19%     
==========================================
  Files          65       63       -2     
  Lines         518      511       -7     
  Branches       31       34       +3     
==========================================
- Hits          443      438       -5     
+ Misses         75       73       -2     

see 5 files with indirect coverage changes

gaeljw commented 8 months ago

I don't understand why 2.12 build fails but for sure it's related to my attempt to not build enumeratum-play for 2.12 anymore. Will investigate a bit more tomorrow. Unless you've got any idea.

Wow.. this was tricky but I think I got it: from what I understand, the commands used in CI were not actually doing what you'd think. The sbt ++<version> cmd1 cmd2 was running only cmd1 with the desired Scala version, cmd2 was running on all Scala versions. Sounds a bit weird, not sure I understood everything to be honest.

This is the reason I had to make changes to CI:

I've done several commits but happy to move these unrelated changes to another PR first if you prefer to.

lloydmeta commented 7 months ago

Thanks for this. Just pushed 1.8.0 of the Play integration libs to Maven central based on this.

gaeljw commented 7 months ago

Great news :)