sandbox/.github/workflows/php-syntaxcheck.yml

---
name: 'PHP Syntax Check'
on:
  push:
    paths:
      - '**.php'
    tags-ignore:
      - 'eslint-config@v*'
      - 'prettier-config@v*'
  pull_request:
  workflow_dispatch:

jobs:
  phpsyntax:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: PHP Basic Syntax Check
        run: php -l ${{ github.workspace }}/PhabExt/PhabricatorCustomRobotsTxtController.php
npm(eslint-config): correct wrong peerDependency Summary: eslint-config-prettier should have been peerDependency not devDependency. Correct them by moving it. (Also do some yml cleanups.) Signed-off-by: Yongmin Hong <revi@omglol.email> Test Plan: `npm ci`. Reviewers: O1 revi & automations, revi Reviewed By: O1 revi & automations, revi Differential Revision: https://issuetracker.revi.xyz/D342 2024-06-17 05:51:51 +02:00			`---`
style: `npm run p:w .` Summary: Which is essentially `npx prettier --write .` Also fix `jshint` (possible) FP, wrong chars. Signed-off-by: Yongmin Hong <revi@omglol.email> Test Plan: `npm run p:w .` and verify no changes made. Reviewers: O1 revi & automations, revi Reviewed By: O1 revi & automations, revi Differential Revision: https://issuetracker.revi.xyz/D217 2024-06-08 12:05:22 +02:00			`name: 'PHP Syntax Check'`
actions: php quick syntax check cf. https://we.phorge.it/book/phorge/article/configuring_preamble/ Bug: N/A Signed-off-by: Yongmin Hong <revi@omglol.email> 2024-04-20 11:04:53 +02:00			`on:`
			`push:`
			`paths:`
			`- '**.php'`
CI(GitHub Actions): rename workflow name/exclude tags Summary: Don't run php check on npm tag push. Signed-off-by: Yongmin Hong <revi@omglol.email> Test Plan: land it and watch when next tagbis uploaded. Reviewers: O1 revi & automations, revi Reviewed By: O1 revi & automations, revi Differential Revision: https://issuetracker.revi.xyz/D448 2024-06-30 10:16:00 +02:00			`tags-ignore:`
			`- 'eslint-config@v*'`
			`- 'prettier-config@v*'`
actions: php quick syntax check cf. https://we.phorge.it/book/phorge/article/configuring_preamble/ Bug: N/A Signed-off-by: Yongmin Hong <revi@omglol.email> 2024-04-20 11:04:53 +02:00			`pull_request:`
			`workflow_dispatch:`

			`jobs:`
			`phpsyntax:`
			`runs-on: ubuntu-latest`
			`steps:`
CI(.arclint): use Pinterest Arcanist Linter Summary: Thanks pinterest! Also fix some GitHub Actions error. Signed-off-by: Yongmin Hong <revi@omglol.email> Test Plan: run `arc lint`? Reviewers: O1 revi & automations, revi Reviewed By: O1 revi & automations, revi Differential Revision: https://issuetracker.revi.xyz/D325 2024-06-16 07:43:28 +02:00			`- name: Checkout`
			`uses: actions/checkout@v4`
Revert "RobotsTxt: specify more verbose classes" Summary: This reverts commit 88946a7fe2e1a5447fca8adbee28f554993be5fe. PHP Fatal error: Cannot override final method PhabricatorRobotsController::processRequest() in /var/www/phorge/phorge/src/extensions/PhabricatorCustomRobotsTxtController.php on line 13 Fatal error: Cannot override final method PhabricatorRobotsController::processRequest() in /var/www/phorge/phorge/src/extensions/PhabricatorCustomRobotsTxtController.php on line 13 Signed-off-by: Yongmin Hong <revi@omglol.email> Reviewers: #acl_sudoers, #blessed_reviewers, revi Reviewed By: #acl_sudoers, #blessed_reviewers, revi Differential Revision: https://issuetracker.revi.xyz/D10 2024-05-02 12:52:47 +02:00			`- name: PHP Basic Syntax Check`
npm(eslint-config): correct wrong peerDependency Summary: eslint-config-prettier should have been peerDependency not devDependency. Correct them by moving it. (Also do some yml cleanups.) Signed-off-by: Yongmin Hong <revi@omglol.email> Test Plan: `npm ci`. Reviewers: O1 revi & automations, revi Reviewed By: O1 revi & automations, revi Differential Revision: https://issuetracker.revi.xyz/D342 2024-06-17 05:51:51 +02:00			`run: php -l ${{ github.workspace }}/PhabExt/PhabricatorCustomRobotsTxtController.php`