Revert #2160 - Githubissues

alan-turing-institute / data-safe-haven

https://data-safe-haven.readthedocs.io

BSD 3-Clause "New" or "Revised" License

61 stars 15 forks source link

Revert #2160 #2232

Closed JimMadge closed 1 month ago

JimMadge commented 1 month ago

:white_check_mark: Checklist

[ ] You have given your pull request a meaningful title (e.g. Enable foobar integration rather than 515 foobar).
[ ] You are targeting the appropriate branch. If you're not certain which one this is, it should be develop.
[ ] Your branch is up-to-date with the target branch (it probably was when you started, but it may have changed since then).

:vertical_traffic_light: Depends on

:arrow_heading_up: Summary

:closed_umbrella: Related issues

:microscope: Tests

github-actions[bot] commented 1 month ago

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
data_safe_haven/commands
sre.py
data_safe_haven/config
context.py
data_safe_haven/external/api
azure_sdk.py					479
graph_api.py					323-349, 1102
data_safe_haven/infrastructure
project_manager.py
data_safe_haven/infrastructure/programs
declarative_sre.py
imperative_shm.py					120
data_safe_haven/provisioning
sre_provisioning_manager.py					76-77, 140
Project Total

This report was generated by python-coverage-comment-action

JimMadge commented 1 month ago

Not needed, errors were due to incompatible state.