Conversation
cd-work
force-pushed
the
1984
branch
2 times, most recently
from
f65535f to
f612ea2
Compare
maxrake
left a comment
maxrake
left a comment
There was a problem hiding this comment.
Merging these changes will introduce failing tests to CI...which is good for test driven development but not so good for ensuring the default branch is in a releasable state.
It is odd that the Windows runner in the test matrix ran and failed somewhat quickly but the Linux and macOS runners are both still running (hung state?), spinning on this new job step.
I did not expect so many tests to fail, but I've fixed all of them in separate commits. |
Because Windows runs into a stack overflow. |
cd-work
force-pushed
the
1984
branch
4 times, most recently
from
285c4a6 to
ecfa0cb
Compare
This is a regression introduced in d7778c1, which changed the subcommand parser to look for `extensions`, instead of the clap-defined `extension`.
This fixes the test for executing extensions and now successfully verifies the output of the sample-extension.
This removes some integration testing attempts which were out of date with the current phylum executable and failing because of it.
This adds a separate test step for all available features, to ensure that the current WIP development features also build without any issue.
4 participants
This adds a separate test step for all available features, to ensure
that the current WIP development features also build without any issue.
Fixes #357.