Unverified Commit 642eb99e authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3529507 by nicxvan, xjm, alexpott: Allow setting memory limit to -1 for... 

Issue #3529507 by nicxvan, xjm, alexpott: Allow setting memory limit to -1 for phpstan precommit hooks

(cherry picked from commit 2ffd3412)
parent f83d7583

core/scripts/dev/commit-code-check.sh

+13 −2
Original line number Diff line number Diff line
@@ -25,6 +25,7 @@ 
  return 1

}



MEMORY_UNLIMITED=0

CACHED=0

DRUPALCI=0

BRANCH=""
@@ -58,12 +59,22 @@ 
      DRUPALCI=1

      shift

      ;;

    --memory-unlimited)

      MEMORY_UNLIMITED=1

      shift

      ;;

    *)

      break

      ;;

  esac

done



memory_limit=""



if [[ "$MEMORY_UNLIMITED" == "1" ]]; then

  memory_limit="--memory-limit=-1"

fi



# Set up variables to make colored output simple. Color output is disabled on

# DrupalCI because it is breaks reporting.

# @todo https://www.drupal.org/project/drupalci_testbot/issues/3181869
@@ -244,11 +255,11 @@ 
# APCu is disabled to ensure that the composer classmap is not corrupted.

if [[ $PHPSTAN_DIST_FILE_CHANGED == "1" ]] || [[ "$DRUPALCI" == "1" ]]; then

  printf "\nRunning PHPStan on *all* files.\n"

  php -d apc.enabled=0 -d apc.enable_cli=0 vendor/bin/phpstan analyze --no-progress --configuration="$TOP_LEVEL/core/phpstan.neon.dist"

  php -d apc.enabled=0 -d apc.enable_cli=0 vendor/bin/phpstan analyze --no-progress --configuration="$TOP_LEVEL/core/phpstan.neon.dist" $memory_limit

else

  # Only run PHPStan on changed files locally.

  printf "\nRunning PHPStan on changed files.\n"

  php -d apc.enabled=0 -d apc.enable_cli=0 vendor/bin/phpstan analyze --no-progress --configuration="$TOP_LEVEL/core/phpstan-partial.neon" $ABS_FILES

  php -d apc.enabled=0 -d apc.enable_cli=0 vendor/bin/phpstan analyze --no-progress --configuration="$TOP_LEVEL/core/phpstan-partial.neon" $ABS_FILES $memory_limit

fi



if [ "$?" -ne "0" ]; then