This follows the idea of keeping the pipeline implementation as dumb as possible.
The (code of the) check can be executed (and hence verified) locally. :heavy_check_mark:
The pipeline implementation is simpler, hence easier to migrate to other CI systems. :heavy_check_mark:
The change also removes a tricky complexity that is hard to investigate and troubleshoot: The version check requires that the package is installed, hence skip_install = true will break the environment. This was not immediately apparent with the original pipeline code, having the two sequential tox calls. The isolated execution of the two tasks/environments makes this more natural to understand.
This follows the idea of keeping the pipeline implementation as dumb as possible.
The change also removes a tricky complexity that is hard to investigate and troubleshoot: The version check requires that the package is installed, hence
skip_install = true
will break the environment. This was not immediately apparent with the original pipeline code, having the two sequentialtox
calls. The isolated execution of the two tasks/environments makes this more natural to understand.