Skip to content

Commit

Permalink
I think this is the correct syntax for not running on draft pr...
Browse files Browse the repository at this point in the history
  • Loading branch information
fcooper8472 committed Feb 22, 2021
1 parent a1be118 commit 29d52f7
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/copyright-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
jobs:

build-and-test:
if: github.event.pull_request.draft == false
name: Copyright
runs-on: ubuntu-latest

Expand Down
1 change: 1 addition & 0 deletions .github/workflows/coverage-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
jobs:

build-and-test:
if: github.event.pull_request.draft == false
name: Coverage
runs-on: ubuntu-latest

Expand Down
1 change: 1 addition & 0 deletions .github/workflows/docs-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
jobs:

build-and-test:
if: github.event.pull_request.draft == false
name: Docs
runs-on: ubuntu-latest

Expand Down
1 change: 1 addition & 0 deletions .github/workflows/style-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
jobs:

build-and-test:
if: github.event.pull_request.draft == false
name: Style
runs-on: ubuntu-latest

Expand Down
1 change: 1 addition & 0 deletions .github/workflows/unit-test-os-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
jobs:

build-and-test:
if: github.event.pull_request.draft == false
name: OS unit tests
runs-on: ${{ matrix.os }}

Expand Down
1 change: 1 addition & 0 deletions .github/workflows/unit-test-python-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
jobs:

build-and-test:
if: github.event.pull_request.draft == false
name: Python unit tests
runs-on: ubuntu-latest

Expand Down

4 comments on commit 29d52f7

@MichaelClerx
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ben18785
Copy link
Collaborator

@ben18785 ben18785 commented on 29d52f7 Feb 22, 2021 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fcooper8472
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It didn't seem to work for me: the previous commit to this, where I merged in the changes from master, still seemed to run everything...

@ben18785
Copy link
Collaborator

@ben18785 ben18785 commented on 29d52f7 Feb 22, 2021 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.