Skip to content

Refactor of the obsid table, since now all obsids are read #29

Refactor of the obsid table, since now all obsids are read

Refactor of the obsid table, since now all obsids are read #29

Triggered via push August 15, 2023 15:28
Status Failure
Total duration 16s
Artifacts

flake8.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors
Ruff (PERF401): acis_thermal_check/acis_obs.py#L304
acis_thermal_check/acis_obs.py:304:13: PERF401 Use a list comprehension to create a transformed list
Ruff (PLR5501): acis_thermal_check/acis_obs.py#L359
acis_thermal_check/acis_obs.py:359:9: PLR5501 Use `elif` instead of `else` then `if`, to reduce indentation
Ruff (PLR5501): acis_thermal_check/apps/acisfp_check.py#L353
acis_thermal_check/apps/acisfp_check.py:353:9: PLR5501 Use `elif` instead of `else` then `if`, to reduce indentation
Ruff (F401): acis_thermal_check/main.py#L27
acis_thermal_check/main.py:27:5: F401 `acis_thermal_check.utils.ValidatePlot` imported but unused
Ruff (COM812): acis_thermal_check/main.py#L1277
acis_thermal_check/main.py:1277:86: COM812 Trailing comma missing
Ruff (D202): acis_thermal_check/main.py#L1281
acis_thermal_check/main.py:1281:9: D202 No blank lines allowed after function docstring (found 1)
Ruff (UP031): acis_thermal_check/main.py#L1321
acis_thermal_check/main.py:1321:13: UP031 Use format specifiers instead of percent format
Ruff (RUF015): acis_thermal_check/state_builder.py#L154
acis_thermal_check/state_builder.py:154:29: RUF015 Prefer `next(...)` over single element slice
Ruff (PLR5501): acis_thermal_check/state_builder.py#L190
acis_thermal_check/state_builder.py:190:9: PLR5501 Use `elif` instead of `else` then `if`, to reduce indentation
Ruff (S301): acis_thermal_check/tests/regression_testing.py#L349
acis_thermal_check/tests/regression_testing.py:349:27: S301 `pickle` and modules that wrap it can be unsafe when used to deserialize untrusted data, possible security issue