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/phpunit-tests.yml

    r54674 r54851  
    3737  # Performs the following steps:
    3838  # - Sets environment variables.
    39   # - Sets up the environment variables needed for testing with memcached (if desired).
    40   # - Installs Node.js.
     39  # - Checks out the repository.
     40  # - Sets up Node.js.
     41  # - Logs general debug information about the runner.
    4142  # - Installs npm dependencies
    4243  # - Configures caching for Composer.
     
    4445  # - Logs Docker debug information (about the Docker installation within the runner).
    4546  # - Starts the WordPress Docker container.
    46   # - Logs general debug information about the runner.
    4747  # - Logs the running Docker containers.
    48   # - Logs debug information from inside the WordPress Docker container.
    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  # - Checks out the WordPress Test reporter repository.
    54   # - Reconnect the directory to the Git repository.
    5553  # - Submit the test results to the WordPress.org host test results.
    5654  test-php:
     
    109107        uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # v3.1.0
    110108
    111       - name: Install Node.js
     109      - name: Set up Node.js
    112110        uses: actions/setup-node@8c91899e586c5b171469028077307d293428b516 # v3.5.1
    113111        with:
     
    115113          cache: npm
    116114
    117       - name: Install Dependencies
     115      - name: General debug information
     116        run: |
     117          npm --version
     118          node --version
     119          curl --version
     120          git --version
     121          svn --version
     122
     123      - name: Install npm dependencies
    118124        run: npm ci
    119125
     
    155161        run: |
    156162          npm run env:start
    157 
    158       - name: General debug information
    159         run: |
    160           npm --version
    161           node --version
    162           curl --version
    163           git --version
    164           svn --version
    165163
    166164      - name: Log running Docker containers
Note: See TracChangeset for help on using the changeset viewer.