refactor: Remove obsolete TODOs (#1048) #645
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: AutoReview | |
| on: | |
| push: | |
| branches: [ main ] | |
| pull_request: ~ | |
| schedule: | |
| # Do not make it the first of the month and/or midnight since it is a very busy time | |
| - cron: "* 10 5 * *" | |
| # See https://stackoverflow.com/a/72408109 | |
| concurrency: | |
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | |
| cancel-in-progress: true | |
| env: | |
| TERM: xterm | |
| jobs: | |
| lint-cs: | |
| runs-on: ubuntu-latest | |
| name: Lint CS | |
| steps: | |
| - name: Checkout | |
| uses: actions/checkout@v3 | |
| - name: Setup PHP | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: '8.1' | |
| tools: composer | |
| coverage: none | |
| - name: Install Composer dependencies | |
| uses: ramsey/composer-install@v2 | |
| - name: Ensure that the make target is up to date | |
| run: make _vendor_install | |
| - name: Install PHP-CS-Fixer | |
| uses: ramsey/composer-install@v2 | |
| with: | |
| working-directory: vendor-bin/php-cs-fixer | |
| - name: Ensure that the PHP-CS-Fixer make target is up to date | |
| run: make php_cs_fixer_install | |
| - run: make root_cs_lint | |
| composer-validate: | |
| runs-on: ubuntu-latest | |
| name: Composer Validate | |
| steps: | |
| - name: Checkout | |
| uses: actions/checkout@v3 | |
| - name: Setup PHP | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: '8.1' | |
| tools: composer | |
| coverage: none | |
| - run: make composer_validate | |
| box-config-validate: | |
| runs-on: ubuntu-latest | |
| name: Box Validate | |
| steps: | |
| - name: Checkout | |
| uses: actions/checkout@v3 | |
| - name: Setup PHP | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: '8.1' | |
| tools: composer | |
| coverage: none | |
| - name: Install Composer dependencies | |
| uses: ramsey/composer-install@v2 | |
| - name: Ensure that the make target is up to date | |
| run: make _vendor_install | |
| - run: make box_validate | |
| autoreview-test: | |
| runs-on: ubuntu-latest | |
| name: AutoReview Tests | |
| steps: | |
| - name: Checkout | |
| uses: actions/checkout@v3 | |
| - name: Setup PHP | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: '8.1' | |
| tools: composer | |
| coverage: none | |
| - name: Install Composer dependencies | |
| uses: ramsey/composer-install@v2 | |
| - name: Ensure that the make target is up to date | |
| run: make _vendor_install | |
| - run: make phpunit_autoreview | |
| # This is a "trick", a meta task which does not change, and we can use in | |
| # the protected branch rules as opposed to the tests one above which | |
| # may change regularly. | |
| validate-autoreview: | |
| name: AutoReview tests status | |
| runs-on: ubuntu-latest | |
| needs: | |
| - lint-cs | |
| - composer-validate | |
| - box-config-validate | |
| - autoreview-test | |
| if: always() | |
| steps: | |
| - name: Successful run | |
| if: ${{ !(contains(needs.*.result, 'failure')) }} | |
| run: exit 0 | |
| - name: Failing run | |
| if: ${{ contains(needs.*.result, 'failure') }} | |
| run: exit 1 |