godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
godot/.github/workflows/static_checks.yml

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

100 lines
4.0 KiB
YAML
Raw Normal View History

GitHub Actions: Prepend emoji to platform names for easy visual grepping
2020-08-22 21:04:57 +00:00
name: 📊 Static Checks
ci: wait for static check results before starting builds
2023-04-04 13:56:53 +00:00
on:
workflow_call:
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
CI: Cancel previous builds if new commit is pushed
2021-08-24 18:49:06 +00:00
concurrency:
Fix CI after concurrency
2021-08-25 13:04:05 +00:00
group: ci-${{github.actor}}-${{github.head_ref || github.run_number}}-${{github.ref}}-static
CI: Cancel previous builds if new commit is pushed
2021-08-24 18:49:06 +00:00
cancel-in-progress: true
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
jobs:
added MacOS and Linux to GitHub Actions
2020-07-14 22:37:02 +00:00
static-checks:
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
name: Code style, file formatting, and docs
runs-on: ubuntu-22.04
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
steps:
- name: Checkout
CI: Bump version for `actions/checkout@v4` and `actions/setup-dotnet@v3`
2023-09-04 14:05:38 +00:00
uses: actions/checkout@v4
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
with:
fetch-depth: 2
- name: Install APT dependencies
uses: awalsh128/cache-apt-pkgs-action@latest
with:
packages: dos2unix libxml2-utils moreutils
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
- name: Install Python dependencies and general setup
CI: Properly exclude Azure repositories on Ubuntu Nuke all the pre-defined repos, we just need stock Ubuntu.
2020-08-11 15:14:47 +00:00
run: |
Replace hand-written pre-commit hooks with `pre-commit` Python tool `pre-commit` can be installed with pip, and configured in the Godot repo with `pre-commit install`. It can then easily be run both locally with `pre-commit run`, and on CI, in a cross-platform way. This makes it much easier for contributors to set up pre-commit hooks, without having to manually copy files to their git folder. Co-authored-by: Rémi Verschelde <rverschelde@gmail.com>
2024-01-09 12:47:16 +00:00
pip3 install pytest==7.1.2 mypy==0.971
CI: Visibly print trailing whitespace when static checks fail GitHub Actions seems to be hiding colored whitespace, and after lots of attempts I couldn't find a way to work it around. So instead I'm using a perl expression to replace trailing spaces with `·` and tabs with `<TAB>` in the ANSI colored diff output. This ensure that they're visible, and they are properly colored as expected too.
2023-04-05 11:51:38 +00:00
git config diff.wsErrorHighlight all
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
- name: Get changed files
id: changed-files
CI: Use gh-cli for changed files, and workaround codespell skip list bug For PRs, this should give a more accurate list, as the previous method would diff to the tip of the `master` branch, which could include new commits (and thus changed files) not present in the PR branch. codespell's `--skip` option doesn't work at all with folders when used together with an explicit list of paths to work with, so let's not use it.
2023-05-08 12:49:51 +00:00
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
run: |
if [ "${{ github.event_name }}" == "pull_request" ]; then
[CI] Improve fetching of changed files PRs always use a merge into the repo on checkout, so checking for `HEAD^1` will show all the changes regardless of the number of commits in a PR
2024-03-28 12:53:32 +00:00
files=$(git diff-tree --no-commit-id --name-only -r HEAD^1..HEAD 2> /dev/null || true)
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
elif [ "${{ github.event_name }}" == "push" -a "${{ github.event.forced }}" == "false" -a "${{ github.event.created }}" == "false" ]; then
CI: Fixup failure to get changed files on merge The checkout might be too shallow so the before commit isn't available. The logic was already written to take this into account (it then generates an empty 'changed.txt' which falls back to testing everything), but the error code would still force terminate the job. Hopefully we can find a way to make the logic work for merge events too in the future, but for now this is a quick fix.
2023-04-20 15:51:53 +00:00
files=$(git diff-tree --no-commit-id --name-only -r ${{ github.event.before }}..${{ github.event.after }} 2> /dev/null || true)
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
fi
echo "$files" >> changed.txt
cat changed.txt
CI: Use gh-cli for changed files, and workaround codespell skip list bug For PRs, this should give a more accurate list, as the previous method would diff to the tip of the `master` branch, which could include new commits (and thus changed files) not present in the PR branch. codespell's `--skip` option doesn't work at all with folders when used together with an explicit list of paths to work with, so let's not use it.
2023-05-08 12:49:51 +00:00
files=$(echo "$files" | grep -v 'thirdparty' | xargs -I {} sh -c 'echo "./{}"' | tr '\n' ' ')
CI: Make codespell checks blocking, but only check changed files
2023-05-08 09:28:34 +00:00
echo "CHANGED_FILES=$files" >> $GITHUB_ENV
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
Add static check for overzealous .gitignores.
2023-07-01 13:32:30 +00:00
# This needs to happen before Python and npm execution; it must happen before any extra files are written.
- name: .gitignore checks (gitignore_check.sh)
run: |
bash ./misc/scripts/gitignore_check.sh
Replace hand-written pre-commit hooks with `pre-commit` Python tool `pre-commit` can be installed with pip, and configured in the Godot repo with `pre-commit install`. It can then easily be run both locally with `pre-commit run`, and on CI, in a cross-platform way. This makes it much easier for contributors to set up pre-commit hooks, without having to manually copy files to their git folder. Co-authored-by: Rémi Verschelde <rverschelde@gmail.com>
2024-01-09 12:47:16 +00:00
- name: Style checks via pre-commit
uses: pre-commit/action@v3.0.1
Buildsystem: Speedup `pre-commit` static check
2024-05-01 14:45:33 +00:00
with:
extra_args: --verbose --files ${{ env.CHANGED_FILES }}
Replace hand-written pre-commit hooks with `pre-commit` Python tool `pre-commit` can be installed with pip, and configured in the Godot repo with `pre-commit install`. It can then easily be run both locally with `pre-commit run`, and on CI, in a cross-platform way. This makes it much easier for contributors to set up pre-commit hooks, without having to manually copy files to their git folder. Co-authored-by: Rémi Verschelde <rverschelde@gmail.com>
2024-01-09 12:47:16 +00:00
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
- name: File formatting checks (file_format.sh)
run: |
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
bash ./misc/scripts/file_format.sh changed.txt
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
Code quality: Fix header guards consistency Adds `header_guards.sh` bash script, used in CI to validate future changes. Can be run locally to fix invalid header guards.
2022-07-23 21:41:51 +00:00
- name: Header guards formatting checks (header_guards.sh)
run: |
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
bash ./misc/scripts/header_guards.sh changed.txt
Code quality: Fix header guards consistency Adds `header_guards.sh` bash script, used in CI to validate future changes. Can be run locally to fix invalid header guards.
2022-07-23 21:41:51 +00:00
ci: add Python static analysis check via mypy
2022-08-23 13:21:46 +00:00
- name: Python scripts static analysis (mypy_check.sh)
run: |
CI: Fix checks fallback logic when changed files listing is missing The file would be one byte (newline) so the check with '-s' failed.
2023-04-20 16:16:38 +00:00
if grep -qE '\.py$|SConstruct|SCsub' changed.txt || [ -z "$(cat changed.txt)" ]; then
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
bash ./misc/scripts/mypy_check.sh
else
echo "Skipping Python static analysis as no Python files were changed."
fi
ci: add Python static analysis check via mypy
2022-08-23 13:21:46 +00:00
ci: add basic test pipeline for shader builders
2022-06-30 02:14:07 +00:00
- name: Python builders checks via pytest (pytest_builders.sh)
run: |
bash ./misc/scripts/pytest_builders.sh
[HTML5] Add jsdoc2rst tool. A template for `jsdoc` that generat the HTML5 public classref. The script can be run via `npm run docs` to print to stdout. You can dry run via `npm run docs -- --d dry-run` or write to file via `npm run docs -- -d /path/to/file.rst` Also update Makefile in `doc/` and add dry run test to CI.
2021-02-22 05:10:16 +00:00
- name: JavaScript style and documentation checks via ESLint and JSDoc
[HTML5] Enforce JavaScript style with eslint. Applies to javascript files inside the platform library folder, the exposed Engine code, and any javascript files in modules. Files ending with ".externs.js" will be ignored, you can create a ".eslintignore" file to specify extra files to be ignored.
2020-11-19 04:59:12 +00:00
run: |
Fix lint errors in WebXR module
2024-02-23 22:22:37 +00:00
if grep -q "\.js" changed.txt || [ -z "$(cat changed.txt)" ]; then
CI: Speed up static checks by checking only changed files - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
2023-04-19 13:10:36 +00:00
cd platform/web
npm ci
npm run lint
npm run docs -- -d dry-run
else
echo "Skipping JavaScript formatting as no Web/JS files were changed."
fi
[HTML5] Enforce JavaScript style with eslint. Applies to javascript files inside the platform library folder, the exposed Engine code, and any javascript files in modules. Files ending with ".externs.js" will be ignored, you can create a ".eslintignore" file to specify extra files to be ignored.
2020-11-19 04:59:12 +00:00
Add an XML schema for documentation This makes it easier to spot syntax errors when editing the class reference. The schema is referenced locally so validation can still work offline. Each class XML's schema conformance is also checked on GitHub Actions.
2022-02-14 13:18:53 +00:00
- name: Class reference schema checks
run: |
[Export docs] Move docs to platform folders.
2023-04-19 09:23:22 +00:00
xmllint --noout --schema doc/class.xsd doc/classes/*.xml modules/*/doc_classes/*.xml platform/*/doc_classes/*.xml
Add an XML schema for documentation This makes it easier to spot syntax errors when editing the class reference. The schema is referenced locally so validation can still work offline. Each class XML's schema conformance is also checked on GitHub Actions.
2022-02-14 13:18:53 +00:00
Add static formatting checks for GitHub Actions
2020-07-13 07:34:34 +00:00
- name: Documentation checks
run: |
Fix `doc_status.py` trying to get removed `version` tag from XML This also runs `doc_status.py` on CI to catch potential future regressions.
2023-07-13 03:20:44 +00:00
doc/tools/doc_status.py doc/classes modules/*/doc_classes platform/*/doc_classes
Add `dotnet format` to CI to check C# style
2022-08-27 01:14:27 +00:00
CI: Add official codespell action with PR annotations
2023-02-01 09:50:52 +00:00
- name: Spell checks via codespell
CI: Skip codespell check if `CHANGED_FILES` is empty This can happen when only thirdparty files are changed, since we grep them out. Re-add `bin` and `thirdparty` to the skip list for good measure.
2023-05-10 07:44:35 +00:00
if: github.event_name == 'pull_request' && env.CHANGED_FILES != ''
Fix various typos with codespell And ignore some false positives introduced by recent versions of codespell.
2023-06-16 06:45:35 +00:00
uses: codespell-project/actions-codespell@v2
CI: Add official codespell action with PR annotations
2023-02-01 09:50:52 +00:00
with:
CI: Skip codespell check if `CHANGED_FILES` is empty This can happen when only thirdparty files are changed, since we grep them out. Re-add `bin` and `thirdparty` to the skip list for good measure.
2023-05-10 07:44:35 +00:00
skip: "./bin,./thirdparty,*.desktop,*.gen.*,*.po,*.pot,*.rc,./AUTHORS.md,./COPYRIGHT.txt,./DONORS.md,./core/input/gamecontrollerdb.txt,./core/string/locales.h,./editor/project_converter_3_to_4.cpp,./misc/scripts/codespell.sh,./platform/android/java/lib/src/com,./platform/web/node_modules,./platform/web/package-lock.json"
Fix various typos with codespell Using 2.2.7.dev217+g10c2abcf. Had to add `colour` to the ignore list as we used it as an alias/keyword for the documentation of color-related APIs. Also ignore recommendations to change `thirdparty` to either `third-party` or `third party`, which are correct but we use the former fairly consistently.
2024-05-07 08:05:42 +00:00
ignore_words_list: "breaked,colour,curvelinear,doubleclick,expct,findn,gird,hel,inout,lod,mis,nd,numer,ot,requestor,te,thirdparty,vai"
CI: Make codespell checks blocking, but only check changed files
2023-05-08 09:28:34 +00:00
path: ${{ env.CHANGED_FILES }}