Upgrade SDK to use ApplyNamespace instead of ApplyScope

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

Upgrade SDK to use ApplyNamespace instead of ApplyScope #183

Closed jaredoconnell closed 1 month ago

jaredoconnell commented 1 month ago

Fixes #179

Changes introduced with this PR

This upgrades the SDK. This switches out a few function calls, and utilizes a convenience type for InvalidSerializationDetector.

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

jaredoconnell commented 1 month ago

I agree that the map of map isn't ideal. I could try a type alias, but I don't think that's worth it. I don't think much is gained by much more abstraction. I reduced redundancy in anyTypeWithExpressions by embedding it. All redundant functions were already removed in InvalidSerializationDetector in a prior commit.