Skip to content

Upgrade to Scala 2.13: cross builds for 2.11 and 2.12, all tests pass - #283

Open
orbang wants to merge 1 commit into
Bayer-Group:masterfrom
orbang:scala-2.13
Open

Upgrade to Scala 2.13: cross builds for 2.11 and 2.12, all tests pass#283
orbang wants to merge 1 commit into
Bayer-Group:masterfrom
orbang:scala-2.13

Conversation

@orbang

@orbang orbang commented Jul 9, 2021

Copy link
Copy Markdown

Hi, I have upgraded cloudformation-template-generator to Scala 2.13. It still cross builds for 2.11 and 2.12. Changes:

  • The majority of changes are related to the MapView change in the collections API
  • ScalaTest patch version upgrade (the old one doesn't have 2.13 artifacts)
  • A workaround has been added for the JsonWritable implicit conversion, which is required because of implicit resolution changes in 2.13 (see comment)
  • Traversable has been removed in Scala 2.13, replaced with an Iterator based impl.

Would it be possible to publish artifacts for 2.13 at some point?


override def iterator: Iterator[A] =
if (parts.isEmpty)
Seq[A]().iterator

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Scala 2.11 doesn't have Iterator.empty...

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.

1 participant