Explicitly use scala.collection.Seq to allow use of mutable collections in Scala 2.13

almond/src/main/scala/plotly/Almond.scala

@@ -4,6 +4,7 @@ import java.lang.{Boolean => JBoolean, Double => JDouble, Integer => JInt}
 
 import almond.interpreter.api.{DisplayData, OutputHandler}
 
+import scala.collection.immutable.Seq
 import scala.util.Random
 import plotly.element._
 import plotly.layout._

core/shared/src/main/scala/plotly/Sequence.scala

@@ -2,6 +2,9 @@ package plotly
 
 import plotly.element.LocalDateTime
 
+import scala.collection.{Seq => BaseScalaSeq}
+import scala.collection.immutable.Seq
+
 sealed abstract class Sequence extends Product with Serializable
 
 object Sequence {
@@ -27,4 +30,28 @@ object Sequence {
     Strings(s)
   implicit def fromDateTimes(seq: Seq[LocalDateTime]): Sequence =
     DateTimes(seq)
+
+  implicit def fromMutableDoubleSeq(s: BaseScalaSeq[Double]): Sequence =
+    Doubles(makeImmutableSeq(s))
+  implicit def fromMutableFloatSeq(s: BaseScalaSeq[Float]): Sequence =
+    Doubles(makeImmutableSeq(s.map(_.toDouble)))
+  implicit def fromMutableIntSeq(s: BaseScalaSeq[Int]): Sequence =
+    Doubles(makeImmutableSeq(s.map(_.toDouble)))
+  implicit def fromMutableLongSeq(s: BaseScalaSeq[Long]): Sequence =
+    Doubles(makeImmutableSeq(s.map(_.toDouble)))
+  implicit def fromMutableNestedDoubleSeq(s: BaseScalaSeq[BaseScalaSeq[Double]]): Sequence =
+    NestedDoubles(makeImmutableSeq(s.map(makeImmutableSeq)))
+  implicit def fromMutableNestedIntSeq(s: BaseScalaSeq[BaseScalaSeq[Int]]): Sequence =
+    NestedInts(makeImmutableSeq(s.map(makeImmutableSeq)))
+  implicit def fromMutableStringSeq(s: BaseScalaSeq[String]): Sequence =
+    Strings(makeImmutableSeq(s))
+  implicit def fromMutableDateTimes(seq: BaseScalaSeq[LocalDateTime]): Sequence =
+    DateTimes(makeImmutableSeq(seq))
+
+  private def makeImmutableSeq[A](mutableSeq: BaseScalaSeq[A]): Seq[A] =
+    mutableSeq match {
+      case asImmutableSeq: Seq[A] => asImmutableSeq
+      case _ => mutableSeq.toVector
+    }
+
 }

core/shared/src/main/scala/plotly/Trace.scala

@@ -1,5 +1,6 @@
 package plotly
 
+import scala.collection.immutable.Seq
 import scala.language.implicitConversions
 
 import java.lang.{ Boolean => JBoolean, Double => JDouble }

core/shared/src/main/scala/plotly/element/ColorScale.scala

@@ -1,5 +1,7 @@
 package plotly.element
+
 import dataclass.data
+import scala.collection.immutable.Seq
 
 sealed abstract class ColorScale extends Product with Serializable
 

core/shared/src/main/scala/plotly/element/Error.scala

@@ -1,6 +1,8 @@
 package plotly
 package element
 
+import scala.collection.immutable.Seq
+
 import dataclass.data
 
 import java.lang.{ Boolean => JBoolean, Double => JDouble }

core/shared/src/main/scala/plotly/element/HoverInfo.scala

@@ -1,5 +1,7 @@
 package plotly.element
 
+import scala.collection.immutable.Seq
+
 sealed abstract class HoverInfo extends Product with Serializable {
   def label: String
 }
@@ -10,7 +12,7 @@ object HoverInfo {
   def none: HoverInfo = None
   def skip: HoverInfo = Skip
   def apply(elements: Element*): HoverInfo =
-    Combination(elements)
+    Combination(Seq(elements: _*))
 
 
   sealed abstract class Element(override val label: String) extends HoverInfo

core/shared/src/main/scala/plotly/element/OneOrSeq.scala

@@ -1,5 +1,7 @@
 package plotly.element
 
+import scala.collection.immutable.Seq
+
 sealed abstract class OneOrSeq[T] extends Product with Serializable
 
 object OneOrSeq {

core/shared/src/main/scala/plotly/layout/Layout.scala

@@ -5,6 +5,7 @@ import java.lang.{ Integer => JInt, Double => JDouble, Boolean => JBoolean }
 
 import plotly.element._
 import dataclass.data
+import scala.collection.immutable.Seq
 
 @data class Layout(
           title: Option[String],

demo/src/main/scala/plotly/demo/DemoChart.scala

@@ -3,6 +3,8 @@ package plotly.demo
 import plotly.Trace
 import plotly.layout.Layout
 
+import scala.collection.immutable.Seq
+
 trait DemoChart {
   def plotlyDocUrl: String
   def id: String

joda-time/src/main/scala/plotly/Joda.scala

@@ -1,5 +1,6 @@
 package plotly
 
+import scala.collection.immutable.Seq
 import org.joda.time._
 import plotly.Sequence.DateTimes
 

render/js/src/main/scala/plotly/Plotly.scala

@@ -9,6 +9,7 @@ import plotly.element.Color
 import plotly.internals.BetterPrinter
 import plotly.layout._
 
+import scala.collection.immutable.Seq
 import scala.scalajs.js
 import scala.scalajs.js.Dynamic.{global => g}
 import scala.scalajs.js.JSON

render/jvm/src/main/scala/plotly/Plotly.scala

@@ -13,6 +13,7 @@ import argonaut.{Json, PrettyParams}
 import plotly.internals.{BetterPrinter, Properties}
 
 import scala.annotation.tailrec
+import scala.collection.immutable.Seq
 
 object Plotly {
 

render/shared/src/main/scala/plotly/Enumerate.scala

@@ -1,5 +1,6 @@
 package plotly
 
+import scala.collection.immutable.Seq
 import shapeless.{ :+:, ::, CNil, Coproduct, Generic, HList, HNil, Inl, Inr, Strict }
 
 sealed abstract class Enumerate[T] {

render/shared/src/main/scala/plotly/internals/ArgonautCodecsExtra.scala

@@ -1,6 +1,7 @@
 package plotly.internals
 
 import argonaut.{DecodeJson, EncodeJson}
+import scala.collection.immutable.Seq
 
 trait ArgonautCodecsExtra {
 

render/shared/src/main/scala/plotly/internals/ArgonautCodecsInternals.scala

@@ -9,6 +9,7 @@ import argonaut.ArgonautShapeless._
 import argonaut.derive._
 import shapeless._
 
+import scala.collection.immutable.Seq
 import scala.util.Try
 import plotly.element._
 import plotly.layout._

tests/src/test/scala/plotly/SequenceTests.scala

@@ -0,0 +1,31 @@
+package plotly
+
+import org.scalatest.FlatSpec
+
+import scala.collection.mutable.ArrayBuffer
+
+class SequenceTests extends FlatSpec {
+
+  "The implicit sequence conversion" should "convert a List to a Sequence" in {
+    assert((List(1, 2, 3): Sequence) === Sequence.Doubles(List(1d, 2d, 3d)))
+  }
+
+  it should "convert a mutable ArrayBuffer to a Sequence" in {
+    assert((ArrayBuffer(1, 2, 3): Sequence) === Sequence.Doubles(List(1d, 2d, 3d)))
+  }
+
+  it should "convert a nested mutable ArrayBuffer to a Sequence" in {
+    val mutableNestedDoubles: ArrayBuffer[ArrayBuffer[Double]] = ArrayBuffer(
+      ArrayBuffer(1d, 2d),
+      ArrayBuffer(3d, 4d),
+    )
+
+    val nestedDoublesList: List[List[Double]] = List(
+      List(1d, 2d),
+      List(3d, 4d),
+    )
+
+    assert((mutableNestedDoubles: Sequence) === Sequence.NestedDoubles(nestedDoublesList))
+  }
+
+}