Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GH Actions: various tweaks #325

Merged
merged 2 commits into from
Dec 19, 2022
Merged

GH Actions: various tweaks #325

merged 2 commits into from
Dec 19, 2022

Commits on Dec 19, 2022

  1. GH Actions: minor simplification 

    ... of the bash `date` command in the earlier pulled cache busting.
    @jrfnl
    jrfnl committed Dec 19, 2022
    Configuration menu
    Copy the full SHA
    cb95c3c View commit details
    Browse the repository at this point in the history

  2. GH Actions: improve performance of the CS step 

    All the repos in the Yoast organisation contain a `<arg name="cache" value="./.cache/phpcs.cache"/>` directive in the PHPCS ruleset.
This directive makes running PHPCS faster by caching the run results in a file and only scanning changed files when running PHPCS again.

However, when there is no cache available, running with the `cache` option enabled will make PHPCS _slower_ as the cache needs to be created and the file read/write actions slow PHPCS down.

In GH Actions, we are not caching the PHPCS `cache` file, which means that there is cache file available and running with `cache` will be slower.

By adding the `--no-cache` option, the `cache` directive in the ruleset is ignored, which should result in a slightly faster runtime for the CS workflow.

Note: the alternative would be to _cache_ the cache file in GH Actions, but aside from the two very frequently changing repos, there's not much point doing that.
    @jrfnl
    jrfnl committed Dec 19, 2022
    Configuration menu
    Copy the full SHA
    76f02b9 View commit details
    Browse the repository at this point in the history