Closed braingram closed 2 weeks ago
dkist added benchmarks to their test suite in https://github.com/DKISTDC/dkist/pull/382
which they disable for unit test runs using --benchmark-skip
This PR adds the same skip to our downstream dkist test (which is otherwise failing on the benchmarks).
[ ] for a public change, added a towncrier news fragment changes/<PR#>.<changetype>.rst
changes/<PR#>.<changetype>.rst
changes/<PR#>.feature.rst
changes/<PR#>.bugfix.rst
changes/<PR#>.doc.rst
changes/<PR#>.removal.rst
changes/<PR#>.general.rst
@Cadair is this the correct way to skip the benchmarks?
I think so yeah.
Description
dkist added benchmarks to their test suite in https://github.com/DKISTDC/dkist/pull/382
which they disable for unit test runs using --benchmark-skip
This PR adds the same skip to our downstream dkist test (which is otherwise failing on the benchmarks).
Checklist:
[ ] for a public change, added a towncrier news fragment
changes/<PR#>.<changetype>.rst
changes/<PR#>.feature.rst
: new featurechanges/<PR#>.bugfix.rst
: bug fixchanges/<PR#>.doc.rst
: documentation changechanges/<PR#>.removal.rst
: deprecation or removal of public APIchanges/<PR#>.general.rst
: infrastructure or miscellaneous change