Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TST: Remove custom CI skip #5734

Merged
merged 1 commit into from
Feb 10, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 1 addition & 12 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -23,19 +23,10 @@ jobs:
cancel_ci:
name: Mandatory checks before CI
runs-on: ubuntu-latest
outputs:
run_next: ${{ steps.skip_ci_step.outputs.run_next }}
steps:
- name: Set output for skip CI
uses: OpenAstronomy/action-skip-ci@main
id: skip_ci_step
with:
NO_FAIL: true
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# This should only run if we did not skip CI
# TODO: Need to fix for pull_request
- name: Cancel Previous Runs
uses: styfle/cancel-workflow-action@ce177499ccf9fd2aded3b0426c97e5434c2e8a73
if: steps.skip_ci_step.outputs.run_next == 'true'
with:
access_token: ${{ secrets.GITHUB_TOKEN }}

@@ -45,7 +36,6 @@ jobs:
name: ${{ matrix.name }}
runs-on: ${{ matrix.os }}
needs: cancel_ci
if: needs.cancel_ci.outputs.run_next == 'true'
strategy:
fail-fast: false
matrix:
@@ -165,7 +155,6 @@ jobs:
# name: ${{ matrix.name }}
# runs-on: ${{ matrix.os }}
# needs: cancel_ci
# if: needs.cancel_ci.outputs.run_next == 'true'
# continue-on-error: ${{ matrix.allowed-failure }}
# strategy:
# fail-fast: false