We have parallelism and python features for many of our crates. "parallelism" can only be compiled in on Linux (not MacOS)? and "python" feature requires build-time and run-time deps. Our CI builds some combinations of these (python, -nopython) but we don't run things like our tests and cargo hecks for all combinations of features. we should see about either removing the parallelism feature, and having only python / no python features, to minimize the different compile-time paths, and/or we should run all of our testing / cargo clippy checks / et al on ALL combinations of our features. This is important because we currently don't run the clippy lintson thepythonfeature, nor do we run any of our tests without theparallelism` feature. It's important that every code path in our repo is considered equal.
We have
parallelismandpythonfeatures for many of our crates. "parallelism" can only be compiled in on Linux (not MacOS)? and "python" feature requires build-time and run-time deps. Our CI builds some combinations of these (python, -nopython) but we don't run things like our tests and cargo hecks for all combinations of features. we should see about either removing theparallelismfeature, and having onlypython/ no python features, to minimize the different compile-time paths, and/or we should run all of our testing / cargo clippy checks / et al on ALL combinations of our features. This is important because we currently don't run theclippylintson thepythonfeature, nor do we run any of our tests without theparallelism` feature. It's important that every code path in our repo is considered equal.