Skip to content

Commit 184b542

Browse files
committed
Remove "Check Datasets" step
This is now superseded by the ehrQL permissions system: * opensafely-core/ehrql#2543
1 parent a7f977e commit 184b542

File tree

6 files changed

+0
-242
lines changed

6 files changed

+0
-242
lines changed

.github/workflows/tests.yml

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -134,34 +134,3 @@ jobs:
134134
echo "Action should have failed with codelists error"
135135
exit 1
136136
fi
137-
138-
test-check:
139-
runs-on: ubuntu-20.04
140-
name: Test dataset check failure
141-
steps:
142-
- name: Checkout
143-
uses: actions/checkout@v4
144-
- name: Test project with failing codelists check
145-
uses: ./
146-
id: dataset
147-
continue-on-error: true
148-
with:
149-
directory: tests/dataset
150-
users: '@nobody'
151-
- name: Assert check failed and created issue
152-
env:
153-
ISSUE_URL: ${{ steps.dataset.outputs.issue }}
154-
run: |
155-
fail() {
156-
echo "$@"
157-
exit 1
158-
}
159-
cleanup() {
160-
gh issue close "$ISSUE_URL" || true
161-
}
162-
trap cleanup EXIT
163-
test "${{ steps.dataset.outcome }}" = "failure" || fail "Action should have failed with dataset error"
164-
test -n "$ISSUE_URL" || fail "No issue url output from dataset step"
165-
gh issue view "$ISSUE_URL" || fail "Could not view issue at $ISSUE_URL"
166-
167-

action.yml

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,6 @@ inputs:
66
description: "Directory containing OpenSAFELY project"
77
required: false
88
default: .
9-
users:
10-
description: "@names of users who should be notified on dataset failure"
11-
required: false
12-
default: "@amirmehrkar @JulietUnderdown1 @LiamHart-hub @alexwalkerepi"
139
actions:
1410
description: "Names of project actions to run during tests (default 'run_all')"
1511
required: false
@@ -32,31 +28,6 @@ runs:
3228
shell: bash
3329
working-directory: ${{ inputs.directory }}
3430
run: opensafely codelists check
35-
- name: Check datasets
36-
shell: bash
37-
id: datasets
38-
working-directory: ${{ inputs.directory }}
39-
run: |
40-
success=true
41-
OUTPUT=$(opensafely check 2>&1) || success=false
42-
echo "$OUTPUT"
43-
# exit early if we're all good
44-
test "$success" = "true" && exit 0
45-
46-
BODY=$(cat << EOF
47-
Dataset check for $GITHUB_REPOSITORY at $GITHUB_SHA:
48-
\`\`\`
49-
$OUTPUT
50-
\`\`\`
51-
${{ inputs.users }}
52-
EOF
53-
)
54-
55-
ISSUE_URL=$(gh issue create --title "Dataset check fail" --body "$BODY")
56-
57-
echo "Created issue at $ISSUE_URL"
58-
echo "::set-output name=issue::$ISSUE_URL"
59-
exit 1
6031
- name: Run project
6132
shell: bash
6233
working-directory: ${{ inputs.directory }}

tests/dataset/analysis/study_definition.py

Lines changed: 0 additions & 170 deletions
This file was deleted.

tests/dataset/logs/.gitkeep

Whitespace-only changes.

tests/dataset/output/.gitkeep

Whitespace-only changes.

tests/dataset/project.yaml

Lines changed: 0 additions & 12 deletions
This file was deleted.

0 commit comments

Comments
 (0)