search

arcalot / arcaflow-engine

Arcaflow is a highly-portable workflow engine enabling modular and validated pipelines through containerized plugins.
https://arcalot.io/arcaflow/
Apache License 2.0
6 stars 9 forks source link

Use double-serialization detector object in test #153

Closed jaredoconnell closed 7 months ago

jaredoconnell commented 7 months ago

Changes introduced with this PR

This is the use of the detector object. It would have detected the bug we found earlier.

By contributing to this repository, I agree to the contribution guidelines.

webbnh commented 7 months ago

(I used the button here to rebase this branch on to the end of main.)

dbutenhof commented 7 months ago

And it's just a bit funny that, despite the rebase and merging your PR into here, your approval checkmark still shows from before ...

webbnh commented 7 months ago

And it's just a bit funny that, despite the rebase and merging your PR into here, your approval checkmark still shows from before ...

Yeah, I think that's a repo setting, but apparently I don't have sufficient priv's here to check.