Skip to content

Commit

Permalink
removing lintr checks for 2.1
Browse files Browse the repository at this point in the history
  • Loading branch information
shaneknapp committed Aug 10, 2018
1 parent 4222943 commit f2974fb
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions dev/run-tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -212,18 +212,6 @@ def run_python_style_checks():
run_cmd([os.path.join(SPARK_HOME, "dev", "lint-python")])


def run_sparkr_style_checks():
set_title_and_block("Running R style checks", "BLOCK_R_STYLE")

if which("R"):
# R style check should be executed after `install-dev.sh`.
# Since warnings about `no visible global function definition` appear
# without the installation. SEE ALSO: SPARK-9121.
run_cmd([os.path.join(SPARK_HOME, "dev", "lint-r")])
else:
print("Ignoring SparkR style check as R was not found in PATH")


def build_spark_documentation():
set_title_and_block("Building Spark Documentation", "BLOCK_DOCUMENTATION")
os.environ["PRODUCTION"] = "1 jekyll build"
Expand Down Expand Up @@ -561,8 +549,6 @@ def main():
pass
if not changed_files or any(f.endswith(".py") for f in changed_files):
run_python_style_checks()
if not changed_files or any(f.endswith(".R") for f in changed_files):
run_sparkr_style_checks()

# determine if docs were changed and if we're inside the amplab environment
# note - the below commented out until *all* Jenkins workers can get `jekyll` installed
Expand Down

0 comments on commit f2974fb

Please sign in to comment.