Matt Vallillo
Matt Vallillo
> thanks @vachillo, this seems to be trickier than what I originally thought. In this case, we'd need to have a single service per Node, correct? That seems a bit...
Also requesting a new feature/workaround for this situation
@torabshaikh thanks for the contribution!
are you imagining the syntax to be something like: ``` class ActivityMixin: allowlist: Optional[list[str] | dict[str, Optional[list[str]]]] = field(default=None, kw_only=True) denylist: Optional[list[str] | dict[str, Optional[list[str]]]] = field(default=None, kw_only=True) ``` where...
> @vachillo all tests appear to be failing missed imports
unit tests you could inspect `config._drivers` ?
> Technically there is no `_drivers` because of the `alias`. I'll remove the attrs magic since users will never create a `_Config` directly. It'll always be used via the setters....
> Ok I really don't know how to test this because of this [root conftest](https://github.com/griptape-ai/griptape/blob/8b9e10c7eeb37e1ddca85c2fd4cc45f2d2595a0a/tests/unit/conftest.py#L17-L21) which immediately initializes the value for all tests. According to Codecov we're already covered here...
> 87.50% of diff hit (target 87.18%) impressive
> > ``` > > * a way to "catch" `ErrorArtifacts` in a Pipeline or Workflow by specifying a Task that will be run only if an `ErrorArtifact` gets returned...