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 c6c80be5..069795f7 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 74ec316a..04b25cf2 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 4c5751ae..5f41c301 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 2cfbab67..ef8debb3 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: -- 2.39.5