force `subprocess.run` to check return code is 0

In general, this should help to capture unintentional errors.

Where the change has created failures, they have been fixed. (Mostly because the git repository that was looked for was missing.)

.gitlab-ci.yml

@@ -77,6 +77,7 @@ style:
     - *prepare_style
     - black --version
     - ruff --version
+    - pydocstyle --version
     - ruff .
     - black --check --diff .
     - pydocstyle mpsd-software-environment.py