bpo-39837 made Azure Pipelines CI non-mandatory. Travis CI was made mandatory to have at least one mandatory job: python/core-workflow#365

I would prefer to have at least one mandatory Windows CI running on Python pull requests. Before we had pre-commit checks, it was too common to break Windows. I would prefer to not regress on that aspect.

--

Currently, GitHub action jobs ignore "documentation-only" changes. Extract of .github/workflows/build.yml:

on:
push:
branches:
- master
- 3.8
- 3.7
paths-ignore:
- 'Doc/'
- 'Misc/'
- '**/.md'
- '**/.rst'

If a job is marked as mandatory but the GitHub action is skipped because of paths-ignore, the PR cannot be merged :-( It seems to be a known GitHub limitation which is not going to be fixed soon:
https://discuss.python.org/t/make-one-windows-ci-job-mandatory/4047/6

To workaround is to always run GitHub action jobs, even on documentation-only jobs.

I propose to always run GitHub actions and then make the Windows (64 bit) job mandatory.

--

Sadly, it will waste resources for documentation-only jobs. To avoid that, the Windows job can be modified later to be skipped if it detects that the PR is a documentation-only change.

Steve Dower wrote:
"That workaround looks roughly like what is in the Azure Pipelines files. Someone just needs the time to go and migrate it."
https://discuss.python.org/t/make-one-windows-ci-job-mandatory/4047/6

Note: the Travis CI job (.travis.yml) uses the following check:

  # Pull requests are slightly complicated because $TRAVIS_COMMIT_RANGE
  # may include more changes than desired if the history is convoluted.
  # Instead, explicitly fetch the base branch and compare against the
  # merge-base commit.
  git fetch -q origin +refs/heads/$TRAVIS_BRANCH
  changes=$(git diff --name-only HEAD $(git merge-base HEAD FETCH_HEAD))
  echo "Files changed:"
  echo "$changes"
  if ! echo "$changes" | grep -qvE '(\.rst$)|(^Doc)|(^Misc)'
  then
    echo "Only docs were updated, stopping build process."
    exit
  fi

--

More background on these CI issues:

• https://bugs.python.org/issue39837
• Make Travis CI (and Windows x64 ?) mandatory core-workflow#365
• https://discuss.python.org/t/make-one-windows-ci-job-mandatory/4047

The workaround presented at https://github.community/t5/GitHub-Actions/Feature-request-conditional-required-checks/m-p/51344/highlight/true#M8157 seems pretty reasonable. And it seems like travis is alerady doing the same thing as it doesn't have native support to skip runs based on path changes.

I can open a PR if you want.

The workaround presented at https://github.community/t5/GitHub-Actions/Feature-request-conditional-required-checks/m-p/51344/highlight/true#M8157 seems pretty reasonable. And it seems like travis is alerady doing the same thing as it doesn't have native support to skip runs based on path changes.

I suppose that Steve Dower mentioned the following part of .azure-pipelines/prebuild-checks.yml:
----------------

• script: |
  if ! git diff --name-only $(diffTarget) | grep -qvE '(\.rst$|^Doc|^Misc)'
  then
  echo "Only docs were updated: tests.run=false"
  echo "##vso[task.setvariable variable=run;isOutput=true]false"
  else
  echo "Code was updated: tests.run=true"
  echo "##vso[task.setvariable variable=run;isOutput=true]true"
  fi
  displayName: Detect source changes
  name: tests
  ----------------

I understood that he proposed to port it inside the Windows job of GitHub actions (.github/workflows/build.yml).

I propose the following plan:

• Always run GitHub Action jobs
• Make Windows 64-bit job mandatory
• And only then try to skip documentation-only changes

• And only then try to skip documentation-only changes

Done :)

New changeset 4e36376 by Victor Stinner in branch 'master':
bpo-40548: Always run GitHub action, even on doc PRs (GH-19981)
4e36376

I propose the following plan:

• Always run GitHub Action jobs

Done.

• Make Windows 64-bit job mandatory

My request: python/core-workflow#368

New changeset 75d7257 by Filipe Laíns in branch 'master':
bpo-40548: GitHub Action workflow: skip jobs on doc only PRs (GH-19983)
75d7257

New changeset 6a78589 by Filipe Laíns in branch 'master':
bpo-40548: github actions: pass the changes check on no source changes (GH-20097)
6a78589

New changeset 07bd5cf by Victor Stinner in branch '3.8':
[3.8] bpo-40548: GitHub Action workflow: skip jobs on doc only PRs (GH-20100)
07bd5cf

New changeset 6ad51a1 by Miss Islington (bot) in branch '3.7':
[3.8] bpo-40548: GitHub Action workflow: skip jobs on doc only PRs (GH-20100)
6ad51a1

New changeset c444108 by Filipe Laíns in branch 'master':
bpo-40548: Github Actions: update actions/checkout to v2 (GH-20164)
c444108

New changeset 9ecf25e by Miss Islington (bot) in branch '3.7':
bpo-40548: Github Actions: update actions/checkout to v2 (GH-20164)
9ecf25e

New changeset d8cbfa2 by Miss Islington (bot) in branch '3.8':
bpo-40548: Github Actions: update actions/checkout to v2 (GH-20164)
d8cbfa2

The initial issue (be able to make a GitHub action job mandatory) is now fixed, I close the issue.

Making Windows x64 job mandatory is now tracked at: python/core-workflow#368

New changeset eaa5517 by Victor Stinner in branch 'master':
bpo-40548: Fix "Check for source changes (pull_request)" GH Action job (GH-21806)
eaa5517

New changeset 09d8260 by Miss Islington (bot) in branch '3.8':
bpo-40548: Fix "Check for source changes (pull_request)" GH Action job (GH-21806)
09d8260

New changeset 2cd58d8 by Miss Islington (bot) in branch '3.9':
bpo-40548: Fix "Check for source changes (pull_request)" GH Action job (GH-21806)
2cd58d8