New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove the inner tag to allow attaching the aspect to layers with multiple dependencies #1322

Merged

vigoo merged 1 commit into zio:master from soujiro32167:remove-inner-tag

Aug 13, 2024

Contributor

soujiro32167 commented Aug 13, 2024

Example:

val callTracing: AwsCallAspect[Tracer] = ???

val aws: ZLayer[AwsConfig, Throwable,  CognitoIdentityProvider] = ???

// Didn't compile before, because of the inner tag requirement
val withAspect:  ZLayer[AwsConfig & Tracer, Throwable,  CognitoIdentityProvider] = aws @@@ callTracing

CLAassistant commented Aug 13, 2024 •

edited

Loading

All committers have signed the CLA.


          Remove the inner tag to allow attaching the aspect to layers with mul…

3ac63bf

…tiple dependencies

soujiro32167 force-pushed the remove-inner-tag branch from a1b4f37 to 3ac63bf Compare

August 13, 2024 17:56

vigoo merged commit a191784 into zio:master

27 checks passed

soujiro32167 deleted the remove-inner-tag branch

August 13, 2024 20:08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet