+1 on what alex said
On Jan 2, 2014 11:04 AM, "Alexandre Bergeron" <notifications@github.com>
wrote:

> In
> commons-gearman/src/it/scala/com/wajam/gearman/GearmanIntegrationTest.scala:
>
> > @@ -0,0 +1,51 @@
> > +package com.wajam.gearman
> > +
> > +import org.scalatest.{BeforeAndAfterAll, FlatSpec}
> > +import org.scalatest.matchers.ShouldMatchers
> > +import org.gearman._
> > +import com.wajam.gearman.utils.GearmanJSON
> > +
> > +class GearmanIntegrationTest extends FlatSpec with ShouldMatchers with BeforeAndAfterAll {
> > +
> > +  protected val TEST_FUNCTION_NAME = "wajam_gearman_it_test"
> > +
> > +  //Create the service
> > +  private val gearmanService: Gearman = new Gearman()
>
> From my point of view, I think it doesn't provide much information.
>
> Scala style guide <http://docs.scala-lang.org/style/types.html#inference>mentions
>
> You should almost never annotate the type of a private field or a local
> variable, as their type will usually be immediately evident in their value.
> [...] However, you may wish to still display the type where the assigned
> value has a complex or non-obvious form.
>
> I don't think this is complex enough to justify specifying the type.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/wajam/scala-commons/pull/29/files#r8614954>
> .
>