chore: fix CI matrix, fix job skipping

[shortcuts]

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/APIC-375

Changes included:

This PR contains a few CI fixes:

• Remove always usage (opened issue: Job-level "if" condition not evaluated correctly if job in "needs" property is skipped actions/runner#491)
  • Skipped jobs does not return success status, we previously used always which is not correct as the CI will keep running jobs even if previous one fails.
• Read CACHE_VERSION from .cache_version file
• Add skips for scripts and codegen
• Fix empty baseRef variable (reading from its own output)

🧪 Test

CI :D

[netlify]

✔️ Deploy Preview for api-clients-automation canceled.

🔨 Explore the source changes: 578be2c

🔍 Inspect the deploy log: https://app.netlify.com/sites/api-clients-automation/deploys/622f56dd869cd8000862830b

[millotp]

Working well ! All the cache key are name 1-blablabla, can you add a description like before
please (1-search-spec-blablabla) ? It's hard to debug otherwise

[shortcuts]

Working well ! All the cache key are name 1-blablabla, can you add a description like before please (1-search-spec-blablabla) ? It's hard to debug otherwise

I've removed description from the key to ease the maintenance of the GitHub actions, it was taking at lot of "visual" space and was not helpful since we have their path on the CI (see screenshot).

Would you still want to see it? It would basically change from

      - name: Cache '${{ matrix.client }}' bundled specs
        id: cache
        uses: actions/cache@v2
        with:
          path: specs/bundled/${{ matrix.client }}.yml
          key: |
            ${{ env.CACHE_VERSION }}-${{
            hashFiles(
              format('specs/{0}/**', matrix.client),
              'specs/common/**'
            )}}

to

      - name: Cache '${{ matrix.client }}' bundled specs
        id: cache
        uses: actions/cache@v2
        with:
          path: specs/bundled/${{ matrix.client }}.yml
          key: |
            ${{ env.CACHE_VERSION }}-${{
              matrix.client
            }}-spec-${{
            hashFiles(
              format('specs/{0}/**', matrix.client),
              'specs/common/**'
            )}}

[millotp]

You don't have to write it on multiple line, it can just be

${{ env.CACHE_VERSION }}-${{ matrix.client }}-spec-${{
  hashFiles(
    format('specs/{0}/**', matrix.client),
    'specs/common/**'
)}}

But I think it will help, in the post restore cache step the path is not written for example

[shortcuts]

But I think it will help, in the post restore cache step the path is not written for example

You still have the context of the job etc. I still think it should not be tackled in this PR as it's a really verbose and repetitive change, it would be easier to review on an other one

[shortcuts]

Created https://algolia.atlassian.net/browse/APIC-377 @millotp