aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorReynold Xin <rxin@databricks.com>2015-04-07 12:37:33 -0700
committerReynold Xin <rxin@databricks.com>2015-04-07 12:37:33 -0700
commit12322159147581602978f7f5a6b33b887ef781a1 (patch)
treec3c0289147d9ffcef589a31f88da27e2aba789e8
parent2c32bef1790dac6f77ef9674f6106c2e24ea0338 (diff)
downloadspark-12322159147581602978f7f5a6b33b887ef781a1.tar.gz
spark-12322159147581602978f7f5a6b33b887ef781a1.tar.bz2
spark-12322159147581602978f7f5a6b33b887ef781a1.zip
[SPARK-6750] Upgrade ScalaStyle to 0.7.
0.7 fixes a bug that's pretty useful, i.e. inline functions no longer return explicit type definition. Author: Reynold Xin <rxin@databricks.com> Closes #5399 from rxin/style0.7 and squashes the following commits: 54c41b2 [Reynold Xin] Actually update the version. 09c759c [Reynold Xin] [SPARK-6750] Upgrade ScalaStyle to 0.7.
-rw-r--r--project/plugins.sbt2
-rw-r--r--project/project/SparkPluginBuild.scala16
-rw-r--r--project/spark-style/src/main/scala/org/apache/spark/scalastyle/NonASCIICharacterChecker.scala39
3 files changed, 2 insertions, 55 deletions
diff --git a/project/plugins.sbt b/project/plugins.sbt
index ee45b6a519..7096b0d3ee 100644
--- a/project/plugins.sbt
+++ b/project/plugins.sbt
@@ -19,7 +19,7 @@ addSbtPlugin("com.github.mpeltonen" % "sbt-idea" % "1.6.0")
addSbtPlugin("net.virtual-void" % "sbt-dependency-graph" % "0.7.4")
-addSbtPlugin("org.scalastyle" %% "scalastyle-sbt-plugin" % "0.6.0")
+addSbtPlugin("org.scalastyle" %% "scalastyle-sbt-plugin" % "0.7.0")
addSbtPlugin("com.typesafe" % "sbt-mima-plugin" % "0.1.6")
diff --git a/project/project/SparkPluginBuild.scala b/project/project/SparkPluginBuild.scala
index 8863f272da..471d00bd82 100644
--- a/project/project/SparkPluginBuild.scala
+++ b/project/project/SparkPluginBuild.scala
@@ -24,20 +24,6 @@ import sbt.Keys._
* becomes available for scalastyle sbt plugin.
*/
object SparkPluginDef extends Build {
- lazy val root = Project("plugins", file(".")) dependsOn(sparkStyle, sbtPomReader)
- lazy val sparkStyle = Project("spark-style", file("spark-style"), settings = styleSettings)
+ lazy val root = Project("plugins", file(".")) dependsOn(sbtPomReader)
lazy val sbtPomReader = uri("https://github.com/ScrapCodes/sbt-pom-reader.git#ignore_artifact_id")
-
- // There is actually no need to publish this artifact.
- def styleSettings = Defaults.defaultSettings ++ Seq (
- name := "spark-style",
- organization := "org.apache.spark",
- scalaVersion := "2.10.4",
- scalacOptions := Seq("-unchecked", "-deprecation"),
- libraryDependencies ++= Dependencies.scalaStyle
- )
-
- object Dependencies {
- val scalaStyle = Seq("org.scalastyle" %% "scalastyle" % "0.4.0")
- }
}
diff --git a/project/spark-style/src/main/scala/org/apache/spark/scalastyle/NonASCIICharacterChecker.scala b/project/spark-style/src/main/scala/org/apache/spark/scalastyle/NonASCIICharacterChecker.scala
deleted file mode 100644
index 3d43c35299..0000000000
--- a/project/spark-style/src/main/scala/org/apache/spark/scalastyle/NonASCIICharacterChecker.scala
+++ /dev/null
@@ -1,39 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-
-package org.apache.spark.scalastyle
-
-import java.util.regex.Pattern
-
-import org.scalastyle.{PositionError, ScalariformChecker, ScalastyleError}
-
-import scalariform.lexer.Token
-import scalariform.parser.CompilationUnit
-
-class NonASCIICharacterChecker extends ScalariformChecker {
- val errorKey: String = "non.ascii.character.disallowed"
-
- override def verify(ast: CompilationUnit): List[ScalastyleError] = {
- ast.tokens.filter(hasNonAsciiChars).map(x => PositionError(x.offset)).toList
- }
-
- private def hasNonAsciiChars(x: Token) =
- x.rawText.trim.nonEmpty && !Pattern.compile( """\p{ASCII}+""", Pattern.DOTALL)
- .matcher(x.text.trim).matches()
-
-}