diff options
author | Massimiliano Culpo <massimiliano.culpo@gmail.com> | 2021-03-16 16:16:31 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-03-16 08:16:31 -0700 |
commit | b304b4bdb0f703569b76696c219670bd2d5dc680 (patch) | |
tree | 16e58d2724f3b29185dfa429bbbc35b3dfef9651 /.github/workflows/style_and_docs.yaml | |
parent | d36de79ba02252ae3279832e85d9f056af475b11 (diff) | |
download | spack-b304b4bdb0f703569b76696c219670bd2d5dc680.tar.gz spack-b304b4bdb0f703569b76696c219670bd2d5dc680.tar.bz2 spack-b304b4bdb0f703569b76696c219670bd2d5dc680.tar.xz spack-b304b4bdb0f703569b76696c219670bd2d5dc680.zip |
Speed-up CI by reorganizing tests (#22247)
* unit tests: mark slow tests as "maybeslow"
This commit also removes the "network" marker and
marks every "network" test as "maybeslow". Tests
marked as db are maintained, but they're not slow
anymore.
* GA: require style tests to pass before running unit-tests
* GA: make MacOS unit tests fail fast
* GA: move all unit tests into the same workflow, run style tests as a prerequisite
All the unit tests have been moved into the same workflow so that a single
run of the dorny/paths-filter action can be used to ask for coverage based
on the files that have been changed in a PR. The basic idea is that for PRs
that introduce only changes to packages coverage is not necessary, this
resulting in a faster execution of the tests.
Also, for package only PRs slow unit tests are skipped.
Finally, MacOS and linux unit tests are now conditional on style tests passing
meaning that e.g. we won't waste a MacOS worker if we know that the PR has
flake8 issues.
* Addressed review comments
* Skipping slow tests on MacOS for package only recipes
* QA: make tests on changes correct before merging
Diffstat (limited to '.github/workflows/style_and_docs.yaml')
-rw-r--r-- | .github/workflows/style_and_docs.yaml | 65 |
1 files changed, 0 insertions, 65 deletions
diff --git a/.github/workflows/style_and_docs.yaml b/.github/workflows/style_and_docs.yaml deleted file mode 100644 index 0f84e1e885..0000000000 --- a/.github/workflows/style_and_docs.yaml +++ /dev/null @@ -1,65 +0,0 @@ -name: style and docs - -on: - push: - branches: - - develop - - releases/** - pull_request: - branches: - - develop - - releases/** -jobs: - validate: - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - - uses: actions/setup-python@v2 - with: - python-version: 3.9 - - name: Install Python Packages - run: | - pip install --upgrade pip - pip install --upgrade vermin - - name: Minimum Version (Spack's Core) - run: vermin --backport argparse --backport typing -t=2.6- -t=3.5- -v lib/spack/spack/ lib/spack/llnl/ bin/ - - name: Minimum Version (Repositories) - run: vermin --backport argparse --backport typing -t=2.6- -t=3.5- -v var/spack/repos - style: - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - with: - fetch-depth: 0 - - uses: actions/setup-python@v2 - with: - python-version: 3.9 - - name: Install Python packages - run: | - pip install --upgrade pip six setuptools flake8 mypy>=0.800 black - - name: Setup git configuration - run: | - # Need this for the git tests to succeed. - git --version - . .github/workflows/setup_git.sh - - name: Run style tests - run: | - share/spack/qa/run-style-tests - documentation: - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - - uses: actions/setup-python@v2 - with: - python-version: 3.9 - - name: Install System packages - run: | - sudo apt-get -y update - sudo apt-get install -y coreutils ninja-build graphviz - - name: Install Python packages - run: | - pip install --upgrade pip six setuptools - pip install --upgrade -r lib/spack/docs/requirements.txt - - name: Build documentation - run: | - share/spack/qa/run-doc-tests |