Skip to content

Commit 5fed000

Browse files
committed
feat: Remove check command and permissions file
This is now superseded by the ehrQL permissions system: * opensafely-core/ehrql#2543 Note that this PR should not be merged until the related `research-action` PR has been merged, which removes the one usage of the `opensafely check` command: * opensafely-core/research-action#115
1 parent 69ed3bd commit 5fed000

File tree

4 files changed

+0
-761
lines changed

4 files changed

+0
-761
lines changed

opensafely/__init__.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
from pathlib import Path
88

99
from opensafely import ( # noqa: E402
10-
check,
1110
clean,
1211
codelists,
1312
execute,
@@ -129,7 +128,6 @@ def add_subcommand(cmd, module):
129128
add_subcommand("codelists", codelists)
130129
add_subcommand("pull", pull)
131130
add_subcommand("upgrade", upgrade)
132-
add_subcommand("check", check)
133131
add_subcommand("unzip", unzip)
134132
add_subcommand("extract-stats", extract_stats)
135133
add_subcommand("info", info)
@@ -160,10 +158,6 @@ def add_subcommand(cmd, module):
160158
print(str(exc), file=sys.stderr)
161159
success = False
162160

163-
# if `run`ning locally, run `check` in warn mode
164-
if function == local_run.main and "format-output-for-github" not in kwargs:
165-
check.main(continue_on_error=True)
166-
167161
# allow functions to return True/False, or an explicit exit code
168162
if success is False:
169163
exit_code = 1

opensafely/check.py

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

repository_permissions.yaml

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

0 commit comments

Comments
 (0)