commit-code-check.sh

Same filename and directory in other branches
  1. 10 core/scripts/dev/commit-code-check.sh
  2. 11.x core/scripts/dev/commit-code-check.sh
  3. 9 core/scripts/dev/commit-code-check.sh
  4. 8.9.x core/scripts/dev/commit-code-check.sh
#!/bin/bash
#
# This script performs code quality checks.
#
# @internal
#   This script is not covered by Drupal core's backwards compatibility promise.
#   It exists only for core development purposes.
#
# The script makes the following checks:
# - Spell checking.
# - File modes.
# - No changes to core/node_modules directory.
# - PHPCS checks PHP and YAML files.
# - PHPStan checks PHP files.
# - ESLint checks JavaScript and YAML files.
# - Stylelint checks CSS files.
# - Checks .pcss.css and .css files are equivalent.

# cSpell:disable

# Searches an array.
contains_element() {
  local e
  for e in ${@:2}; do [[ "$e" == "$1" ]] && return 0; done
  return 1
}

MEMORY_UNLIMITED=0
CACHED=0
DRUPALCI=0
BRANCH=""
while test $# -gt 0; do
  case "$1" in
    -h|--help)
      echo "Drupal code quality checks"
      echo " "
      echo "options:"
      echo "-h, --help                show brief help"
      echo "--branch BRANCH           creates list of files to check by comparing against a branch"
      echo "--cached                  checks staged files"
      echo "--drupalci                a special mode for DrupalCI"
      echo "--memory-unlimited        bypass PHP memory limit for PHPStan and PHPCS"
      echo " "
      echo "Example usage: sh ./core/scripts/dev/commit-code-check.sh --branch 9.2.x"
      exit 0
      ;;
    --branch)
      BRANCH="$2"
      if [[ "$BRANCH" == "" ]]; then
        printf "The --branch option requires a value. For example: --branch 9.2.x\n"
        exit;
      fi
      shift 2
      ;;
    --cached)
      CACHED=1
      shift
      ;;
    --drupalci)
      DRUPALCI=1
      shift
      ;;
    --memory-unlimited)
      MEMORY_UNLIMITED=1
      shift
      ;;
    *)
      break
      ;;
  esac
done

memory_limit=""
phpcs_memory_limit=""

if [[ "$MEMORY_UNLIMITED" == "1" ]]; then
  memory_limit="--memory-limit=-1"
  phpcs_memory_limit="-d 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
if [[ "$DRUPALCI" == "1" ]]; then
  red=""
  green=""
  reset=""
  DRUPAL_VERSION=$(php -r "include 'vendor/autoload.php'; print preg_replace('#\.[0-9]+-dev#', '.x', \Drupal::VERSION);")
  GIT="sudo -u www-data git"
else
  red=$(tput setaf 1 && tput bold)
  blue=$(tput setaf 4 && tput bold)
  green=$(tput setaf 2)
  reset=$(tput sgr0)
  GIT="git"
fi

# Gets list of files to check.
if [[ "$BRANCH" != "" ]]; then
  FILES=$($GIT diff --name-only $BRANCH HEAD);
elif [[ "$CACHED" == "0" ]]; then
  # For DrupalCI patch testing or when running without --cached or --branch,
  # list of all changes in the working directory.
  FILES=$($GIT ls-files --other --modified --exclude-standard --exclude=vendor)
else
  # Check staged files only.
  if $GIT rev-parse --verify HEAD >/dev/null 2>&1
  then
    AGAINST=HEAD
  else
    # Initial commit: diff against an empty tree object
    AGAINST=4b825dc642cb6eb9a060e54bf8d69288fbee4904
  fi
  FILES=$($GIT diff --cached --name-only $AGAINST);
fi

if [[ "$FILES" == "" ]] && [[ "$DRUPALCI" == "1" ]]; then
  # If the FILES is empty we might be testing a merge request on DrupalCI. We
  # need to diff against the Drupal branch or tag related to the Drupal version.
  printf "Creating list of files to check by comparing branch to %s\n" "$DRUPAL_VERSION"
  # On DrupalCI there's a merge commit so we can compare to HEAD~1.
  FILES=$($GIT diff --name-only HEAD~1 HEAD);
fi

TOP_LEVEL=$($GIT rev-parse --show-toplevel)

# This variable will be set to one when the file core/phpcs.xml.dist is changed.
PHPCS_XML_DIST_FILE_CHANGED=0

# This variable will be set to one when the files core/.phpstan-baseline.php or
# core/phpstan.neon.dist are changed.
PHPSTAN_DIST_FILE_CHANGED=0

# This variable will be set to one when one of the eslint config file is
# changed:
#  - core/.eslintrc.passing.json
#  - core/.eslintrc.json
#  - core/.eslintrc.jquery.json
ESLINT_CONFIG_PASSING_FILE_CHANGED=0

# This variable will be set to one when the stylelint config file is changed.
# changed:
#  - core/.stylelintignore
#  - core/.stylelintrc.json
STYLELINT_CONFIG_FILE_CHANGED=0

# This variable will be set to one when JavaScript packages files are changed.
# changed:
#  - core/package.json
#  - core/yarn.lock
JAVASCRIPT_PACKAGES_CHANGED=0

# This variable will be set when a Drupal-specific CKEditor 5 plugin has changed
# it is used to make sure the compiled JS is valid.
CKEDITOR5_PLUGINS_CHANGED=0

# This variable will be set to one when either of the core dictionaries or the
# .cspell.json config has changed.
CSPELL_DICTIONARY_FILE_CHANGED=0

# Build up a list of absolute file names.
ABS_FILES=
for FILE in $FILES; do
  if [ -f "$TOP_LEVEL/$FILE" ]; then
    ABS_FILES="$ABS_FILES $TOP_LEVEL/$FILE"
  fi

  if [[ $FILE == "core/phpcs.xml.dist" ]]; then
    PHPCS_XML_DIST_FILE_CHANGED=1;
  fi;

  if [[ $FILE == "core/.phpstan-baseline.php" || $FILE == "core/phpstan.neon.dist" ]]; then
    PHPSTAN_DIST_FILE_CHANGED=1;
  fi;

  if [[ $FILE == "core/.eslintrc.json" || $FILE == "core/.eslintrc.passing.json" || $FILE == "core/.eslintrc.jquery.json" ]]; then
    ESLINT_CONFIG_PASSING_FILE_CHANGED=1;
  fi;

  if [[ $FILE == "core/.stylelintignore" || $FILE == "core/.stylelintrc.json" ]]; then
    STYLELINT_CONFIG_FILE_CHANGED=1;
  fi;

  # If JavaScript packages change, then rerun all JavaScript style checks.
  if [[ $FILE == "core/package.json" || $FILE == "core/yarn.lock" ]]; then
    ESLINT_CONFIG_PASSING_FILE_CHANGED=1;
    STYLELINT_CONFIG_FILE_CHANGED=1;
    JAVASCRIPT_PACKAGES_CHANGED=1;
  fi;

  if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.js$ ]] && [[ $FILE =~ ^core/modules/ckeditor5/js/build || $FILE =~ ^core/modules/ckeditor5/js/ckeditor5_plugins ]]; then
    CKEDITOR5_PLUGINS_CHANGED=1;
  fi;

  if [[ $FILE == "core/misc/cspell/dictionary.txt" || $FILE == "core/misc/cspell/drupal-dictionary.txt" || $FILE == "core/.cspell.json" ]]; then
    CSPELL_DICTIONARY_FILE_CHANGED=1;
  fi

  if [[ $FILE == "core/MAINTAINERS.txt" ]]; then
    MAINTAINERS_TXT_CHANGED=1;
  fi

done

# Exit early if there are no files.
if [[ "$ABS_FILES" == "" ]]; then
  printf "There are no files to check. If you have staged a commit use the --cached option.\n"
  exit;
fi;

# This script assumes that composer install and yarn install have already been
# run and all dependencies are updated.
FINAL_STATUS=0

DEPENDENCIES_NEED_INSTALLING=0
# Ensure PHP development dependencies are installed.
# @todo https://github.com/composer/composer/issues/4497 Improve this to
#  determine if dependencies in the lock file match the installed versions.
#  Using composer install --dry-run is not valid because it would depend on
#  user-facing strings in Composer.
if ! [[ -f 'vendor/bin/phpcs' ]]; then
  printf "Drupal's PHP development dependencies are not installed. Run 'composer install' from the root directory.\n"
  DEPENDENCIES_NEED_INSTALLING=1;
fi

cd "$TOP_LEVEL/core"

# Ensure JavaScript development dependencies are installed.
yarn --version
yarn >/dev/null

# Check all files for spelling in one go for better performance.
if [[ $CSPELL_DICTIONARY_FILE_CHANGED == "1" ]] ; then
  printf "\nRunning spellcheck on *all* files.\n"
  yarn run spellcheck:core --no-must-find-files --no-progress
else
  # Check all files for spelling in one go for better performance. We pipe the
  # list files in so we obey the globs set on the spellcheck:core command in
  # core/package.json.
  echo "${ABS_FILES}" | tr ' ' '\n' | yarn run spellcheck:core --no-must-find-files --file-list stdin
fi

if [ "$?" -ne "0" ]; then
  # If there are failures set the status to a number other than 0.
  FINAL_STATUS=1
  printf "\nCSpell: ${red}failed${reset}\n"
else
  printf "\nCSpell: ${green}passed${reset}\n"
fi
cd "$TOP_LEVEL"

# Add a separator line to make the output easier to read.
printf "\n"
printf -- '-%.0s' {1..100}
printf "\n"

# Run PHPStan on all files on DrupalCI or when phpstan files are changed.
# 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" $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 $memory_limit
fi

if [ "$?" -ne "0" ]; then
  # If there are failures set the status to a number other than 0.
  FINAL_STATUS=1
  printf "\nPHPStan: ${red}failed${reset}\n"
else
  printf "\nPHPStan: ${green}passed${reset}\n"
fi

# Add a separator line to make the output easier to read.
printf "\n"
printf -- '-%.0s' {1..100}
printf "\n"

# Run PHPCS on all files on DrupalCI or when phpcs files are changed.
if [[ $PHPCS_XML_DIST_FILE_CHANGED == "1" ]] || [[ "$DRUPALCI" == "1" ]]; then
  # Test all files with phpcs rules.
  vendor/bin/phpcs $phpcs_memory_limit -ps --parallel="$( (nproc || sysctl -n hw.logicalcpu || echo 4) 2>/dev/null)" --standard="$TOP_LEVEL/core/phpcs.xml.dist"
  PHPCS=$?
  if [ "$PHPCS" -ne "0" ]; then
    # If there are failures set the status to a number other than 0.
    FINAL_STATUS=1
    printf "\nPHPCS: ${red}failed${reset}\n"
  else
    printf "\nPHPCS: ${green}passed${reset}\n"
  fi
  # Add a separator line to make the output easier to read.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
fi

# When the eslint config has been changed, then eslint must check all files.
if [[ $ESLINT_CONFIG_PASSING_FILE_CHANGED == "1" ]]; then
  cd "$TOP_LEVEL/core"
  yarn run lint:core-js-passing "$TOP_LEVEL/core"
  CORRECTJS=$?
  if [ "$CORRECTJS" -ne "0" ]; then
    # If there are failures set the status to a number other than 0.
    FINAL_STATUS=1
    printf "\neslint: ${red}failed${reset}\n"
  else
    printf "\neslint: ${green}passed${reset}\n"
  fi
  cd $TOP_LEVEL
  # Add a separator line to make the output easier to read.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
fi

# When the stylelint config has been changed, then stylelint must check all files.
if [[ $STYLELINT_CONFIG_FILE_CHANGED == "1" ]]; then
  cd "$TOP_LEVEL/core"
  yarn run lint:css
  if [ "$?" -ne "0" ]; then
    # If there are failures set the status to a number other than 0.
    FINAL_STATUS=1
    printf "\nstylelint: ${red}failed${reset}\n"
  else
    printf "\nstylelint: ${green}passed${reset}\n"
  fi
  cd $TOP_LEVEL
  # Add a separator line to make the output easier to read.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
fi

# When a Drupal-specific CKEditor 5 plugin changed ensure that it is compiled
# properly. Only check on DrupalCI, since we're concerned about the build being
# run with the expected package versions and making sure the result of the build
# is in sync and conform to expectations.
if [[ "$DRUPALCI" == "1" ]] && [[ $CKEDITOR5_PLUGINS_CHANGED == "1" ]]; then
  cd "$TOP_LEVEL/core"
  yarn run check:ckeditor5
  if [ "$?" -ne "0" ]; then
    # If there are failures set the status to a number other than 0.
    FINAL_STATUS=1
    printf "\nDrupal-specific CKEditor 5 plugins: ${red}failed${reset}\n"
  else
    printf "\nDrupal-specific CKEditor 5 plugins: ${green}passed${reset}\n"
  fi
  cd $TOP_LEVEL
  # Add a separator line to make the output easier to read.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
fi

# When JavaScript packages change, then rerun all JavaScript style checks.
if [[ "$JAVASCRIPT_PACKAGES_CHANGED" == "1" ]]; then
  cd "$TOP_LEVEL/core"
  yarn run build:css --check
  CORRECTCSS=$?
  if [ "$CORRECTCSS" -ne "0" ]; then
    FINAL_STATUS=1
    printf "\n${red}ERROR: The compiled CSS from the PCSS files"
    printf "\n       does not match the current CSS files. Some added"
    printf "\n       or updated JavaScript package made changes."
    printf "\n       Recompile the CSS with: yarn run build:css${reset}\n\n"
  fi
  cd $TOP_LEVEL
  # Add a separator line to make the output easier to read.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
fi

for FILE in $FILES; do
  STATUS=0;
  # Print a line to separate spellcheck output from per file output.
  printf "Checking %s\n" "$FILE"
  printf "\n"

  # Ensure the file still exists (i.e. is not being deleted).
  if [ -a $FILE ]; then
    if [ ${FILE: -3} != ".sh" ]; then
      if [ -x $FILE ]; then
        printf "${red}check failed:${reset} file $FILE should not be executable\n"
        STATUS=1
      fi
    fi
  fi

  # Don't commit changes to vendor.
  if [[ "$FILE" =~ ^vendor/ ]]; then
    printf "${red}check failed:${reset} file in vendor directory being committed ($FILE)\n"
    STATUS=1
  fi

  # Don't commit changes to core/node_modules.
  if [[ "$FILE" =~ ^core/node_modules/ ]]; then
    printf "${red}check failed:${reset} file in core/node_modules directory being committed ($FILE)\n"
    STATUS=1
  fi

  ############################################################################
  ### PHP AND YAML FILES
  ############################################################################
  if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.(inc|install|module|php|profile|test|theme|yml)$ ]] && [[ $PHPCS_XML_DIST_FILE_CHANGED == "0" ]] && [[ "$DRUPALCI" == "0" ]]; then
    # Test files with phpcs rules.
    vendor/bin/phpcs $phpcs_memory_limit "$TOP_LEVEL/$FILE" --standard="$TOP_LEVEL/core/phpcs.xml.dist"
    PHPCS=$?
    if [ "$PHPCS" -ne "0" ]; then
      # If there are failures set the status to a number other than 0.
      STATUS=1
    else
      printf "PHPCS: $FILE ${green}passed${reset}\n"
    fi
  fi

  ############################################################################
  ### YAML FILES
  ############################################################################
  if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.yml$ ]]; then
    # Test files with ESLint.
    cd "$TOP_LEVEL/core"
    node ./node_modules/eslint/bin/eslint.js --quiet --resolve-plugins-relative-to . "$TOP_LEVEL/$FILE"
    YAMLLINT=$?
    if [ "$YAMLLINT" -ne "0" ]; then
      # If there are failures set the status to a number other than 0.
      STATUS=1
    else
      printf "ESLint: $FILE ${green}passed${reset}\n"
    fi
    cd $TOP_LEVEL
  fi

  ############################################################################
  ### JAVASCRIPT FILES
  ############################################################################
  if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.js$ ]]; then
    cd "$TOP_LEVEL/core"
    # Check the coding standards.
    node ./node_modules/eslint/bin/eslint.js --quiet --config=.eslintrc.passing.json "$TOP_LEVEL/$FILE"
    JSLINT=$?
    if [ "$JSLINT" -ne "0" ]; then
      # No need to write any output the node command will do this for us.
      STATUS=1
    else
      printf "ESLint: $FILE ${green}passed${reset}\n"
    fi
    cd $TOP_LEVEL
  fi

  ############################################################################
  ### CSS FILES
  ############################################################################
  if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.css$ ]]; then
    # Work out the root name of the CSS so we can ensure that the PostCSS
    # version has been compiled correctly.
    if [[ $FILE =~ \.pcss\.css$ ]]; then
      BASENAME=${FILE%.pcss.css}
      COMPILE_CHECK=1
    else
      BASENAME=${FILE%.css}
      # We only need to compile check if the .pcss.css file is not also
      # changing. This is because the compile check will occur for the
      # .pcss.css file. This might occur if the compiled stylesheets have
      # changed.
      contains_element "$BASENAME.pcss.css" "${FILES[@]}"
      HASPOSTCSS=$?
      if [ "$HASPOSTCSS" -ne "0" ]; then
        COMPILE_CHECK=1
      else
        COMPILE_CHECK=0
      fi
    fi
    # PostCSS
    if [[ "$COMPILE_CHECK" == "1" ]] && [[ -f "$TOP_LEVEL/$BASENAME.pcss.css" ]]; then
      cd "$TOP_LEVEL/core"
      yarn run build:css --check --file "$TOP_LEVEL/$BASENAME.pcss.css"
      CORRECTCSS=$?
      if [ "$CORRECTCSS" -ne "0" ]; then
        # If the CSS does not match the PCSS, set the status to a number other
        # than 0.
        STATUS=1
        printf "\n${red}ERROR: The compiled CSS from"
        printf "\n       ${BASENAME}.pcss.css"
        printf "\n       does not match its CSS file. Recompile the CSS with:"
        printf "\n       yarn run build:css${reset}\n\n"
      fi
      cd $TOP_LEVEL
    fi
  fi
  if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.css$ ]] && [[ -f "core/node_modules/.bin/stylelint" ]]; then
    BASENAME=${FILE%.css}
    # We only need to use stylelint on the .pcss.css file. So if this CSS file
    # has a corresponding .pcss don't do stylelint.
    if [[ $FILE =~ \.pcss\.css$ ]] || [[ ! -f "$TOP_LEVEL/$BASENAME.pcss.css" ]]; then
      cd "$TOP_LEVEL/core"
      node_modules/.bin/stylelint --allow-empty-input "$TOP_LEVEL/$FILE"
      if [ "$?" -ne "0" ]; then
        STATUS=1
      else
        printf "STYLELINT: $FILE ${green}passed${reset}\n"
      fi
      cd $TOP_LEVEL
    fi
  fi

  if [[ "$STATUS" == "1" ]]; then
    FINAL_STATUS=1
    # There is no need to print a failure message. The fail will be described
    # already.
  else
    printf "%s ${green}passed${reset}\n" "$FILE"
  fi

  # Print a line to separate each file's checks.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
done

if [[ "$MAINTAINERS_TXT_CHANGED" == "1" ]]; then
  printf "\n${blue}INFO: MAINTAINERS.TXT changed"
  printf "\n      Make sure follow up changes are made to documentation, Slack channel, email group etc."
  printf "\n      See https://www.drupal.org/about/core/policies/maintainers/add-or-remove-a-subsystem-or-topic-maintainer.${reset}\n\n"

  # Add a separator line to make the output easier to read.
  printf "\n"
  printf -- '-%.0s' {1..100}
  printf "\n"
fi

if [[ "$FINAL_STATUS" == "1" ]] && [[ "$DRUPALCI" == "1" ]]; then
  printf "${red}Drupal code quality checks failed.${reset}\n"
  printf "To reproduce this output locally:\n"
  printf "* Apply the change as a patch\n"
  printf "* Run this command locally: sh ./core/scripts/dev/commit-code-check.sh\n"
  printf "OR:\n"
  printf "* From the merge request branch\n"
  printf "* Run this command locally: sh ./core/scripts/dev/commit-code-check.sh --branch %s\n" "$DRUPAL_VERSION"
fi
exit $FINAL_STATUS

File

View on git.drupalcode.org

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

View source 
  1. #!/bin/bash

  2. #

  3. # This script performs code quality checks.

  4. #

  5. # @internal

  6. #   This script is not covered by Drupal core's backwards compatibility promise.

  7. #   It exists only for core development purposes.

  8. #

  9. # The script makes the following checks:

  10. # - Spell checking.

  11. # - File modes.

  12. # - No changes to core/node_modules directory.

  13. # - PHPCS checks PHP and YAML files.

  14. # - PHPStan checks PHP files.

  15. # - ESLint checks JavaScript and YAML files.

  16. # - Stylelint checks CSS files.

  17. # - Checks .pcss.css and .css files are equivalent.

  18. 

  19. # cSpell:disable

  20. 

  21. # Searches an array.

  22. contains_element() {

  23.   local e

  24.   for e in ${@:2}; do [[ "$e" == "$1" ]] && return 0; done

  25.   return 1

  26. }

  27. 

  28. MEMORY_UNLIMITED=0

  29. CACHED=0

  30. DRUPALCI=0

  31. BRANCH=""

  32. while test $# -gt 0; do

  33.   case "$1" in

  34.     -h|--help)

  35.       echo "Drupal code quality checks"

  36.       echo " "

  37.       echo "options:"

  38.       echo "-h, --help                show brief help"

  39.       echo "--branch BRANCH           creates list of files to check by comparing against a branch"

  40.       echo "--cached                  checks staged files"

  41.       echo "--drupalci                a special mode for DrupalCI"

  42.       echo "--memory-unlimited        bypass PHP memory limit for PHPStan and PHPCS"

  43.       echo " "

  44.       echo "Example usage: sh ./core/scripts/dev/commit-code-check.sh --branch 9.2.x"

  45.       exit 0

  46.       ;;

  47.     --branch)

  48.       BRANCH="$2"

  49.       if [[ "$BRANCH" == "" ]]; then

  50.         printf "The --branch option requires a value. For example: --branch 9.2.x\n"

  51.         exit;

  52.       fi

  53.       shift 2

  54.       ;;

  55.     --cached)

  56.       CACHED=1

  57.       shift

  58.       ;;

  59.     --drupalci)

  60.       DRUPALCI=1

  61.       shift

  62.       ;;

  63.     --memory-unlimited)

  64.       MEMORY_UNLIMITED=1

  65.       shift

  66.       ;;

  67.     *)

  68.       break

  69.       ;;

  70.   esac

  71. done

  72. 

  73. memory_limit=""

  74. phpcs_memory_limit=""

  75. 

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

  77.   memory_limit="--memory-limit=-1"

  78.   phpcs_memory_limit="-d memory_limit=-1"

  79. fi

  80. 

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

  82. # DrupalCI because it is breaks reporting.

  83. # @todo https://www.drupal.org/project/drupalci_testbot/issues/3181869

  84. if [[ "$DRUPALCI" == "1" ]]; then

  85.   red=""

  86.   green=""

  87.   reset=""

  88.   DRUPAL_VERSION=$(php -r "include 'vendor/autoload.php'; print preg_replace('#\.[0-9]+-dev#', '.x', \Drupal::VERSION);")

  89.   GIT="sudo -u www-data git"

  90. else

  91.   red=$(tput setaf 1 && tput bold)

  92.   blue=$(tput setaf 4 && tput bold)

  93.   green=$(tput setaf 2)

  94.   reset=$(tput sgr0)

  95.   GIT="git"

  96. fi

  97. 

  98. # Gets list of files to check.

  99. if [[ "$BRANCH" != "" ]]; then

  100.   FILES=$($GIT diff --name-only $BRANCH HEAD);

  101. elif [[ "$CACHED" == "0" ]]; then

  102.   # For DrupalCI patch testing or when running without --cached or --branch,

  103.   # list of all changes in the working directory.

  104.   FILES=$($GIT ls-files --other --modified --exclude-standard --exclude=vendor)

  105. else

  106.   # Check staged files only.

  107.   if $GIT rev-parse --verify HEAD >/dev/null 2>&1

  108.   then

  109.     AGAINST=HEAD

  110.   else

  111.     # Initial commit: diff against an empty tree object

  112.     AGAINST=4b825dc642cb6eb9a060e54bf8d69288fbee4904

  113.   fi

  114.   FILES=$($GIT diff --cached --name-only $AGAINST);

  115. fi

  116. 

  117. if [[ "$FILES" == "" ]] && [[ "$DRUPALCI" == "1" ]]; then

  118.   # If the FILES is empty we might be testing a merge request on DrupalCI. We

  119.   # need to diff against the Drupal branch or tag related to the Drupal version.

  120.   printf "Creating list of files to check by comparing branch to %s\n" "$DRUPAL_VERSION"

  121.   # On DrupalCI there's a merge commit so we can compare to HEAD~1.

  122.   FILES=$($GIT diff --name-only HEAD~1 HEAD);

  123. fi

  124. 

  125. TOP_LEVEL=$($GIT rev-parse --show-toplevel)

  126. 

  127. # This variable will be set to one when the file core/phpcs.xml.dist is changed.

  128. PHPCS_XML_DIST_FILE_CHANGED=0

  129. 

  130. # This variable will be set to one when the files core/.phpstan-baseline.php or

  131. # core/phpstan.neon.dist are changed.

  132. PHPSTAN_DIST_FILE_CHANGED=0

  133. 

  134. # This variable will be set to one when one of the eslint config file is

  135. # changed:

  136. #  - core/.eslintrc.passing.json

  137. #  - core/.eslintrc.json

  138. #  - core/.eslintrc.jquery.json

  139. ESLINT_CONFIG_PASSING_FILE_CHANGED=0

  140. 

  141. # This variable will be set to one when the stylelint config file is changed.

  142. # changed:

  143. #  - core/.stylelintignore

  144. #  - core/.stylelintrc.json

  145. STYLELINT_CONFIG_FILE_CHANGED=0

  146. 

  147. # This variable will be set to one when JavaScript packages files are changed.

  148. # changed:

  149. #  - core/package.json

  150. #  - core/yarn.lock

  151. JAVASCRIPT_PACKAGES_CHANGED=0

  152. 

  153. # This variable will be set when a Drupal-specific CKEditor 5 plugin has changed

  154. # it is used to make sure the compiled JS is valid.

  155. CKEDITOR5_PLUGINS_CHANGED=0

  156. 

  157. # This variable will be set to one when either of the core dictionaries or the

  158. # .cspell.json config has changed.

  159. CSPELL_DICTIONARY_FILE_CHANGED=0

  160. 

  161. # Build up a list of absolute file names.

  162. ABS_FILES=

  163. for FILE in $FILES; do

  164.   if [ -f "$TOP_LEVEL/$FILE" ]; then

  165.     ABS_FILES="$ABS_FILES $TOP_LEVEL/$FILE"

  166.   fi

  167. 

  168.   if [[ $FILE == "core/phpcs.xml.dist" ]]; then

  169.     PHPCS_XML_DIST_FILE_CHANGED=1;

  170.   fi;

  171. 

  172.   if [[ $FILE == "core/.phpstan-baseline.php" || $FILE == "core/phpstan.neon.dist" ]]; then

  173.     PHPSTAN_DIST_FILE_CHANGED=1;

  174.   fi;

  175. 

  176.   if [[ $FILE == "core/.eslintrc.json" || $FILE == "core/.eslintrc.passing.json" || $FILE == "core/.eslintrc.jquery.json" ]]; then

  177.     ESLINT_CONFIG_PASSING_FILE_CHANGED=1;

  178.   fi;

  179. 

  180.   if [[ $FILE == "core/.stylelintignore" || $FILE == "core/.stylelintrc.json" ]]; then

  181.     STYLELINT_CONFIG_FILE_CHANGED=1;

  182.   fi;

  183. 

  184.   # If JavaScript packages change, then rerun all JavaScript style checks.

  185.   if [[ $FILE == "core/package.json" || $FILE == "core/yarn.lock" ]]; then

  186.     ESLINT_CONFIG_PASSING_FILE_CHANGED=1;

  187.     STYLELINT_CONFIG_FILE_CHANGED=1;

  188.     JAVASCRIPT_PACKAGES_CHANGED=1;

  189.   fi;

  190. 

  191.   if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.js$ ]] && [[ $FILE =~ ^core/modules/ckeditor5/js/build || $FILE =~ ^core/modules/ckeditor5/js/ckeditor5_plugins ]]; then

  192.     CKEDITOR5_PLUGINS_CHANGED=1;

  193.   fi;

  194. 

  195.   if [[ $FILE == "core/misc/cspell/dictionary.txt" || $FILE == "core/misc/cspell/drupal-dictionary.txt" || $FILE == "core/.cspell.json" ]]; then

  196.     CSPELL_DICTIONARY_FILE_CHANGED=1;

  197.   fi

  198. 

  199.   if [[ $FILE == "core/MAINTAINERS.txt" ]]; then

  200.     MAINTAINERS_TXT_CHANGED=1;

  201.   fi

  202. 

  203. done

  204. 

  205. # Exit early if there are no files.

  206. if [[ "$ABS_FILES" == "" ]]; then

  207.   printf "There are no files to check. If you have staged a commit use the --cached option.\n"

  208.   exit;

  209. fi;

  210. 

  211. # This script assumes that composer install and yarn install have already been

  212. # run and all dependencies are updated.

  213. FINAL_STATUS=0

  214. 

  215. DEPENDENCIES_NEED_INSTALLING=0

  216. # Ensure PHP development dependencies are installed.

  217. # @todo https://github.com/composer/composer/issues/4497 Improve this to

  218. #  determine if dependencies in the lock file match the installed versions.

  219. #  Using composer install --dry-run is not valid because it would depend on

  220. #  user-facing strings in Composer.

  221. if ! [[ -f 'vendor/bin/phpcs' ]]; then

  222.   printf "Drupal's PHP development dependencies are not installed. Run 'composer install' from the root directory.\n"

  223.   DEPENDENCIES_NEED_INSTALLING=1;

  224. fi

  225. 

  226. cd "$TOP_LEVEL/core"

  227. 

  228. # Ensure JavaScript development dependencies are installed.

  229. yarn --version

  230. yarn >/dev/null

  231. 

  232. # Check all files for spelling in one go for better performance.

  233. if [[ $CSPELL_DICTIONARY_FILE_CHANGED == "1" ]] ; then

  234.   printf "\nRunning spellcheck on *all* files.\n"

  235.   yarn run spellcheck:core --no-must-find-files --no-progress

  236. else

  237.   # Check all files for spelling in one go for better performance. We pipe the

  238.   # list files in so we obey the globs set on the spellcheck:core command in

  239.   # core/package.json.

  240.   echo "${ABS_FILES}" | tr ' ' '\n' | yarn run spellcheck:core --no-must-find-files --file-list stdin

  241. fi

  242. 

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

  244.   # If there are failures set the status to a number other than 0.

  245.   FINAL_STATUS=1

  246.   printf "\nCSpell: ${red}failed${reset}\n"

  247. else

  248.   printf "\nCSpell: ${green}passed${reset}\n"

  249. fi

  250. cd "$TOP_LEVEL"

  251. 

  252. # Add a separator line to make the output easier to read.

  253. printf "\n"

  254. printf -- '-%.0s' {1..100}

  255. printf "\n"

  256. 

  257. # Run PHPStan on all files on DrupalCI or when phpstan files are changed.

  258. # APCu is disabled to ensure that the composer classmap is not corrupted.

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

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

  261.   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

  262. else

  263.   # Only run PHPStan on changed files locally.

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

  265.   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

  266. fi

  267. 

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

  269.   # If there are failures set the status to a number other than 0.

  270.   FINAL_STATUS=1

  271.   printf "\nPHPStan: ${red}failed${reset}\n"

  272. else

  273.   printf "\nPHPStan: ${green}passed${reset}\n"

  274. fi

  275. 

  276. # Add a separator line to make the output easier to read.

  277. printf "\n"

  278. printf -- '-%.0s' {1..100}

  279. printf "\n"

  280. 

  281. # Run PHPCS on all files on DrupalCI or when phpcs files are changed.

  282. if [[ $PHPCS_XML_DIST_FILE_CHANGED == "1" ]] || [[ "$DRUPALCI" == "1" ]]; then

  283.   # Test all files with phpcs rules.

  284.   vendor/bin/phpcs $phpcs_memory_limit -ps --parallel="$( (nproc || sysctl -n hw.logicalcpu || echo 4) 2>/dev/null)" --standard="$TOP_LEVEL/core/phpcs.xml.dist"

  285.   PHPCS=$?

  286.   if [ "$PHPCS" -ne "0" ]; then

  287.     # If there are failures set the status to a number other than 0.

  288.     FINAL_STATUS=1

  289.     printf "\nPHPCS: ${red}failed${reset}\n"

  290.   else

  291.     printf "\nPHPCS: ${green}passed${reset}\n"

  292.   fi

  293.   # Add a separator line to make the output easier to read.

  294.   printf "\n"

  295.   printf -- '-%.0s' {1..100}

  296.   printf "\n"

  297. fi

  298. 

  299. # When the eslint config has been changed, then eslint must check all files.

  300. if [[ $ESLINT_CONFIG_PASSING_FILE_CHANGED == "1" ]]; then

  301.   cd "$TOP_LEVEL/core"

  302.   yarn run lint:core-js-passing "$TOP_LEVEL/core"

  303.   CORRECTJS=$?

  304.   if [ "$CORRECTJS" -ne "0" ]; then

  305.     # If there are failures set the status to a number other than 0.

  306.     FINAL_STATUS=1

  307.     printf "\neslint: ${red}failed${reset}\n"

  308.   else

  309.     printf "\neslint: ${green}passed${reset}\n"

  310.   fi

  311.   cd $TOP_LEVEL

  312.   # Add a separator line to make the output easier to read.

  313.   printf "\n"

  314.   printf -- '-%.0s' {1..100}

  315.   printf "\n"

  316. fi

  317. 

  318. # When the stylelint config has been changed, then stylelint must check all files.

  319. if [[ $STYLELINT_CONFIG_FILE_CHANGED == "1" ]]; then

  320.   cd "$TOP_LEVEL/core"

  321.   yarn run lint:css

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

  323.     # If there are failures set the status to a number other than 0.

  324.     FINAL_STATUS=1

  325.     printf "\nstylelint: ${red}failed${reset}\n"

  326.   else

  327.     printf "\nstylelint: ${green}passed${reset}\n"

  328.   fi

  329.   cd $TOP_LEVEL

  330.   # Add a separator line to make the output easier to read.

  331.   printf "\n"

  332.   printf -- '-%.0s' {1..100}

  333.   printf "\n"

  334. fi

  335. 

  336. # When a Drupal-specific CKEditor 5 plugin changed ensure that it is compiled

  337. # properly. Only check on DrupalCI, since we're concerned about the build being

  338. # run with the expected package versions and making sure the result of the build

  339. # is in sync and conform to expectations.

  340. if [[ "$DRUPALCI" == "1" ]] && [[ $CKEDITOR5_PLUGINS_CHANGED == "1" ]]; then

  341.   cd "$TOP_LEVEL/core"

  342.   yarn run check:ckeditor5

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

  344.     # If there are failures set the status to a number other than 0.

  345.     FINAL_STATUS=1

  346.     printf "\nDrupal-specific CKEditor 5 plugins: ${red}failed${reset}\n"

  347.   else

  348.     printf "\nDrupal-specific CKEditor 5 plugins: ${green}passed${reset}\n"

  349.   fi

  350.   cd $TOP_LEVEL

  351.   # Add a separator line to make the output easier to read.

  352.   printf "\n"

  353.   printf -- '-%.0s' {1..100}

  354.   printf "\n"

  355. fi

  356. 

  357. # When JavaScript packages change, then rerun all JavaScript style checks.

  358. if [[ "$JAVASCRIPT_PACKAGES_CHANGED" == "1" ]]; then

  359.   cd "$TOP_LEVEL/core"

  360.   yarn run build:css --check

  361.   CORRECTCSS=$?

  362.   if [ "$CORRECTCSS" -ne "0" ]; then

  363.     FINAL_STATUS=1

  364.     printf "\n${red}ERROR: The compiled CSS from the PCSS files"

  365.     printf "\n       does not match the current CSS files. Some added"

  366.     printf "\n       or updated JavaScript package made changes."

  367.     printf "\n       Recompile the CSS with: yarn run build:css${reset}\n\n"

  368.   fi

  369.   cd $TOP_LEVEL

  370.   # Add a separator line to make the output easier to read.

  371.   printf "\n"

  372.   printf -- '-%.0s' {1..100}

  373.   printf "\n"

  374. fi

  375. 

  376. for FILE in $FILES; do

  377.   STATUS=0;

  378.   # Print a line to separate spellcheck output from per file output.

  379.   printf "Checking %s\n" "$FILE"

  380.   printf "\n"

  381. 

  382.   # Ensure the file still exists (i.e. is not being deleted).

  383.   if [ -a $FILE ]; then

  384.     if [ ${FILE: -3} != ".sh" ]; then

  385.       if [ -x $FILE ]; then

  386.         printf "${red}check failed:${reset} file $FILE should not be executable\n"

  387.         STATUS=1

  388.       fi

  389.     fi

  390.   fi

  391. 

  392.   # Don't commit changes to vendor.

  393.   if [[ "$FILE" =~ ^vendor/ ]]; then

  394.     printf "${red}check failed:${reset} file in vendor directory being committed ($FILE)\n"

  395.     STATUS=1

  396.   fi

  397. 

  398.   # Don't commit changes to core/node_modules.

  399.   if [[ "$FILE" =~ ^core/node_modules/ ]]; then

  400.     printf "${red}check failed:${reset} file in core/node_modules directory being committed ($FILE)\n"

  401.     STATUS=1

  402.   fi

  403. 

  404.   ############################################################################

  405.   ### PHP AND YAML FILES

  406.   ############################################################################

  407.   if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.(inc|install|module|php|profile|test|theme|yml)$ ]] && [[ $PHPCS_XML_DIST_FILE_CHANGED == "0" ]] && [[ "$DRUPALCI" == "0" ]]; then

  408.     # Test files with phpcs rules.

  409.     vendor/bin/phpcs $phpcs_memory_limit "$TOP_LEVEL/$FILE" --standard="$TOP_LEVEL/core/phpcs.xml.dist"

  410.     PHPCS=$?

  411.     if [ "$PHPCS" -ne "0" ]; then

  412.       # If there are failures set the status to a number other than 0.

  413.       STATUS=1

  414.     else

  415.       printf "PHPCS: $FILE ${green}passed${reset}\n"

  416.     fi

  417.   fi

  418. 

  419.   ############################################################################

  420.   ### YAML FILES

  421.   ############################################################################

  422.   if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.yml$ ]]; then

  423.     # Test files with ESLint.

  424.     cd "$TOP_LEVEL/core"

  425.     node ./node_modules/eslint/bin/eslint.js --quiet --resolve-plugins-relative-to . "$TOP_LEVEL/$FILE"

  426.     YAMLLINT=$?

  427.     if [ "$YAMLLINT" -ne "0" ]; then

  428.       # If there are failures set the status to a number other than 0.

  429.       STATUS=1

  430.     else

  431.       printf "ESLint: $FILE ${green}passed${reset}\n"

  432.     fi

  433.     cd $TOP_LEVEL

  434.   fi

  435. 

  436.   ############################################################################

  437.   ### JAVASCRIPT FILES

  438.   ############################################################################

  439.   if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.js$ ]]; then

  440.     cd "$TOP_LEVEL/core"

  441.     # Check the coding standards.

  442.     node ./node_modules/eslint/bin/eslint.js --quiet --config=.eslintrc.passing.json "$TOP_LEVEL/$FILE"

  443.     JSLINT=$?

  444.     if [ "$JSLINT" -ne "0" ]; then

  445.       # No need to write any output the node command will do this for us.

  446.       STATUS=1

  447.     else

  448.       printf "ESLint: $FILE ${green}passed${reset}\n"

  449.     fi

  450.     cd $TOP_LEVEL

  451.   fi

  452. 

  453.   ############################################################################

  454.   ### CSS FILES

  455.   ############################################################################

  456.   if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.css$ ]]; then

  457.     # Work out the root name of the CSS so we can ensure that the PostCSS

  458.     # version has been compiled correctly.

  459.     if [[ $FILE =~ \.pcss\.css$ ]]; then

  460.       BASENAME=${FILE%.pcss.css}

  461.       COMPILE_CHECK=1

  462.     else

  463.       BASENAME=${FILE%.css}

  464.       # We only need to compile check if the .pcss.css file is not also

  465.       # changing. This is because the compile check will occur for the

  466.       # .pcss.css file. This might occur if the compiled stylesheets have

  467.       # changed.

  468.       contains_element "$BASENAME.pcss.css" "${FILES[@]}"

  469.       HASPOSTCSS=$?

  470.       if [ "$HASPOSTCSS" -ne "0" ]; then

  471.         COMPILE_CHECK=1

  472.       else

  473.         COMPILE_CHECK=0

  474.       fi

  475.     fi

  476.     # PostCSS

  477.     if [[ "$COMPILE_CHECK" == "1" ]] && [[ -f "$TOP_LEVEL/$BASENAME.pcss.css" ]]; then

  478.       cd "$TOP_LEVEL/core"

  479.       yarn run build:css --check --file "$TOP_LEVEL/$BASENAME.pcss.css"

  480.       CORRECTCSS=$?

  481.       if [ "$CORRECTCSS" -ne "0" ]; then

  482.         # If the CSS does not match the PCSS, set the status to a number other

  483.         # than 0.

  484.         STATUS=1

  485.         printf "\n${red}ERROR: The compiled CSS from"

  486.         printf "\n       ${BASENAME}.pcss.css"

  487.         printf "\n       does not match its CSS file. Recompile the CSS with:"

  488.         printf "\n       yarn run build:css${reset}\n\n"

  489.       fi

  490.       cd $TOP_LEVEL

  491.     fi

  492.   fi

  493.   if [[ -f "$TOP_LEVEL/$FILE" ]] && [[ $FILE =~ \.css$ ]] && [[ -f "core/node_modules/.bin/stylelint" ]]; then

  494.     BASENAME=${FILE%.css}

  495.     # We only need to use stylelint on the .pcss.css file. So if this CSS file

  496.     # has a corresponding .pcss don't do stylelint.

  497.     if [[ $FILE =~ \.pcss\.css$ ]] || [[ ! -f "$TOP_LEVEL/$BASENAME.pcss.css" ]]; then

  498.       cd "$TOP_LEVEL/core"

  499.       node_modules/.bin/stylelint --allow-empty-input "$TOP_LEVEL/$FILE"

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

  501.         STATUS=1

  502.       else

  503.         printf "STYLELINT: $FILE ${green}passed${reset}\n"

  504.       fi

  505.       cd $TOP_LEVEL

  506.     fi

  507.   fi

  508. 

  509.   if [[ "$STATUS" == "1" ]]; then

  510.     FINAL_STATUS=1

  511.     # There is no need to print a failure message. The fail will be described

  512.     # already.

  513.   else

  514.     printf "%s ${green}passed${reset}\n" "$FILE"

  515.   fi

  516. 

  517.   # Print a line to separate each file's checks.

  518.   printf "\n"

  519.   printf -- '-%.0s' {1..100}

  520.   printf "\n"

  521. done

  522. 

  523. if [[ "$MAINTAINERS_TXT_CHANGED" == "1" ]]; then

  524.   printf "\n${blue}INFO: MAINTAINERS.TXT changed"

  525.   printf "\n      Make sure follow up changes are made to documentation, Slack channel, email group etc."

  526.   printf "\n      See https://www.drupal.org/about/core/policies/maintainers/add-or-remove-a-subsystem-or-topic-maintainer.${reset}\n\n"

  527. 

  528.   # Add a separator line to make the output easier to read.

  529.   printf "\n"

  530.   printf -- '-%.0s' {1..100}

  531.   printf "\n"

  532. fi

  533. 

  534. if [[ "$FINAL_STATUS" == "1" ]] && [[ "$DRUPALCI" == "1" ]]; then

  535.   printf "${red}Drupal code quality checks failed.${reset}\n"

  536.   printf "To reproduce this output locally:\n"

  537.   printf "* Apply the change as a patch\n"

  538.   printf "* Run this command locally: sh ./core/scripts/dev/commit-code-check.sh\n"

  539.   printf "OR:\n"

  540.   printf "* From the merge request branch\n"

  541.   printf "* Run this command locally: sh ./core/scripts/dev/commit-code-check.sh --branch %s\n" "$DRUPAL_VERSION"

  542. fi

  543. exit $FINAL_STATUS

Buggy or inaccurate documentation? Please file an issue. Need support? Need help programming? Connect with the Drupal community.