From 3ef339b2e75468a09d617e6aa74bc920c317bce6 Mon Sep 17 00:00:00 2001 From: Richard Si <63936253+ichard26@users.noreply.github.com> Date: Wed, 12 May 2021 21:28:41 -0400 Subject: [PATCH] Modify when Test, Primer, and Documentation Build run (#2226) - Test and Primer don't run for documentation only changes since it's unnecessary, eating unnecessary cycles and slowing down CI since these workflows eat up the 20 max workers limit quite easily! - Documentation Build runs all of the time now since quite a bit of the content depends on Black's code so even a simple 1-file change in src/black/__init__.py may break the docs build. It's not like this is a costly workflow anyway. Fuzz is still running on all changes because with fuzzing, the more the better in general. 6 or 7 jobs on a documentation only commit is much better than 27/28 jobs anyway :p I also found an error in our bug report issue template :) --- .github/ISSUE_TEMPLATE/bug_report.md | 2 +- .github/workflows/doc.yml | 16 +--------------- .github/workflows/primer.yml | 11 ++++++++++- .github/workflows/test.yml | 11 ++++++++++- 4 files changed, 22 insertions(+), 18 deletions(-) diff --git a/.github/ISSUE_TEMPLATE/bug_report.md b/.github/ISSUE_TEMPLATE/bug_report.md index c6c80be5345..069795f7776 100644 --- a/.github/ISSUE_TEMPLATE/bug_report.md +++ b/.github/ISSUE_TEMPLATE/bug_report.md @@ -30,7 +30,7 @@ assignees: "" - clone this repository; - run `pip install -e .[d,python2]`; - run `pip install -r test_requirements.txt` - - make sure it's sane by running `python -m unittest`; and + - make sure it's sane by running `python -m pytest`; and - run `black` like you did last time. **Additional context** Add any other context about the problem here. diff --git a/.github/workflows/doc.yml b/.github/workflows/doc.yml index 74ec316a7cf..04b25cf2a16 100644 --- a/.github/workflows/doc.yml +++ b/.github/workflows/doc.yml @@ -1,20 +1,6 @@ name: Documentation Build -on: - push: - paths: - - "docs/**" - - "README.md" - - "CHANGES.md" - - "CONTRIBUTING.md" - - "AUTHORS.md" - pull_request: - paths: - - "docs/**" - - "README.md" - - "CHANGES.md" - - "CONTRIBUTING.md" - - "AUTHORS.md" +on: [push, pull_request] jobs: build: diff --git a/.github/workflows/primer.yml b/.github/workflows/primer.yml index 4c5751ae996..5f41c301737 100644 --- a/.github/workflows/primer.yml +++ b/.github/workflows/primer.yml @@ -1,6 +1,15 @@ name: Primer -on: [push, pull_request] +on: + push: + paths-ignore: + - "docs/**" + - "*.md" + + pull_request: + paths-ignore: + - "docs/**" + - "*.md" jobs: build: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 2cfbab67ce1..ef8debb3fb7 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,6 +1,15 @@ name: Test -on: [push, pull_request] +on: + push: + paths-ignore: + - "docs/**" + - "*.md" + + pull_request: + paths-ignore: + - "docs/**" + - "*.md" jobs: build: