Make WordPress Core


Ignore:
Timestamp:
11/16/2022 07:32:57 PM (20 months ago)
Author:
desrosj
Message:

Build/Test Tools: Various minor GitHub Action improvements.

This applies several types of improvements to GitHub Action workflows:

  • Updates to inline documentation to ensure accuracy.
  • Removal of repetitive or unnecessary debug logging.
  • Reorganization of some steps to have configuration steps towards the beginning of jobs.
  • Step name updates for consistency across workflows.

Props desrosj, jrf.
See #56793.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/.github/workflows/test-coverage.yml

    r54674 r54851  
    3333
    3434jobs:
    35   # Sets up WordPress for testing or development use.
     35  # Runs the PHPUnit tests for WordPress.
    3636  #
    3737  # Performs the following steps:
    3838  # - Sets environment variables.
    3939  # - Checks out the repository.
    40   # - Checks out the WordPress Importer plugin (needed for the Core PHPUnit tests).
    41   # - Logs debug information about the GitHub Action runner.
    42   # - Installs Node.js.
    43   # _ Installs npm dependencies.
     40  # - Sets up Node.js.
     41  # - Logs general debug information about the runner.
     42  # - Installs npm dependencies
     43  # - Configures caching for Composer.
     44  # - Installs Composer dependencies.
    4445  # - Logs Docker debug information (about the Docker installation within the runner).
    4546  # - Starts the WordPress Docker container.
    46   # - Logs debug general information.
    4747  # - Logs the running Docker containers.
    48   # - Logs WordPress Docker container debug information.
    4948  # - Logs debug information about what's installed within the WordPress Docker containers.
    5049  # - Install WordPress within the Docker container.
     
    5251  # - Ensures version-controlled files are not modified or deleted.
    5352  # - Upload the single site code coverage report to Codecov.io.
    54   # - Run the PHPUnit tests as a multisite.
     53  # - Run the PHPUnit tests as a multisite installation.
    5554  # - Ensures version-controlled files are not modified or deleted.
    5655  # - Upload the multisite code coverage report to Codecov.io.
     
    7473        uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # v3.1.0
    7574
     75      - name: Set up Node.js
     76        uses: actions/setup-node@8c91899e586c5b171469028077307d293428b516 # v3.5.1
     77        with:
     78          node-version-file: '.nvmrc'
     79          cache: npm
     80
    7681      - name: Log debug information
    7782        run: |
     
    8388          git --version
    8489          svn --version
    85           php --version
    86           php -i
    8790          locale -a
    8891
    89       - name: Install Node.js
    90         uses: actions/setup-node@8c91899e586c5b171469028077307d293428b516 # v3.5.1
    91         with:
    92           node-version-file: '.nvmrc'
    93           cache: npm
    94 
    95       - name: Install Dependencies
     92      - name: Install npm Dependencies
    9693        run: npm ci
    9794
     
    129126        run: |
    130127          npm run env:start
    131 
    132       - name: General debug information
    133         run: |
    134           npm --version
    135           node --version
    136           curl --version
    137           git --version
    138           svn --version
    139128
    140129      - name: Log running Docker containers
Note: See TracChangeset for help on using the changeset viewer.