[split] scrooge: Separate flow for linter.

RB_ID=379299
[split] scrooge-linter: Fix multiple arguments to linter + pants/mvn fixes.

RB_ID=382791
[split] bump util to 6.18.1-SNAPSHOT

RB_ID=391383
TBR=true
NO-QUEUE=true
[split] bump finagle to 6.18.1-SNAPSHOT

RB_ID=391531
TBR=true
NO-QUEUE=true
[split] bump scrooge to 3.16.2-SNAPSHOT

RB_ID=391597
TBR=true
NO-QUEUE=true
[split] Normalize scalatest versions between poms and 3rdparty

We use scalatest_2.9.2 version 1.9.1 in 3rdparty, and that's what gets
run with pants tests.  This changes the poms to also reflect this.  This
is needed to unbreak some stuff I'm doing in util wrt testing libs.

RB_ID=392553
[split] scrooge: Fix error message for missing required field.

RB_ID=394667
[split] bump util to 6.18.2-SNAPSHOT

RB_ID=401255
TBR=true
[split] Birdcage: Strip scala mangling from target names

Including the scala mangling in target names causes redundancy and conflicts during scala version upgrades, and is generally not useful. This review strips them.

- Strip scala mangling from target names (while preserving it for jar definitions)
- Remove orphaned com.twitter#json targets

RB_ID=401527
[split] bump finagle to 6.19.1-SNAPSHOT

RB_ID=401701
TBR=true
[split] Rm former employees from OWNERS files of core libraries

RB_ID=401377
TBR=true
[split] Birdcage: Fix some maven parent relative paths

RB_ID=402595
TBR=true
[split] Scrooge: stricter checks for invalid thrift filenames  - A thrift file can be created with characters such as dash in the filename. However this is not allowed when you include that file in another thrift file. This branch adds a check so that this situation cannot arise by validating the filename against the regex used in the latter case.

See CORESTORAGE-6275, THRIFT-196 and CORESTORAGE-6728

RB_ID=402687
[split] Thrift: rename all thrift files with dash in the name  - The dash and other invalid characters mean they cannot be included and referenced in other thrift files.  - For more info: https://jira.twitter.biz/browse/THRIFT-196

RB_ID=403377
[split] scrooge-ostrich: default to using Protocols.binaryFactory

finagle-thrift's `Protocols.binaryFactory` is our drop-in replacement
for `TBinaryProtocol.Factory`.
See 9a0e3c363c9a9ebbb18404248b7254d06e55dd14 for details.

RB_ID=410467
[split] csl: upgraded sbt to 0.13 across the board

RB_ID=410647
[split] bump util to 6.18.3-SNAPSHOT

RB_ID=412425
TBR=true
[split] scrooge: Add union metadata for reflection  Changes in generated code:  *** Added "ThriftUnion" type tag  *** Added key/value manifest references in the *Aliases objects  *** Added "*FieldManifest" and "val fieldInfo = new ThriftStructFieldInfo" for each constructor.

RB_ID=410163
[split] Modify compiler to accept a Scaladoc comment at the end of Thrift file

Signed-off-by: Nik Shkrob <nshkrob@twitter.com>

RB_ID=417203
[split] scrooge: Clarify docs on CLI usage.

RB_ID=419331