Skip to content
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.

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

429 propagate context to reneder param type correctly #433

Conversation

myazinn
Copy link
Collaborator

@myazinn myazinn commented Apr 25, 2023

Fixes #429
Though I have no idea how to write a test for it ¯\_(ツ)_/¯ It looks like it breaks only when package object is placed in a separate file

@myazinn myazinn force-pushed the 429_propagate_context_to_reneder_param_type_correctly branch from 7ab14af to 6196cac Compare April 25, 2023 18:34
@@ -43,71 +43,71 @@ abstract class ModulePatternAccessiblePolyTestBase(injectAlias: Boolean) extends

def test_generates_accessor_function_for_field_of_the_polymorphic_service(): Unit =
assertEquals(
s"def v[R, T <: Example.Foo]: zio.ZIO[$aliasOrHasService with R, Throwable, T] = " +
s"def v[R, T <: Foo]: zio.ZIO[$aliasOrHasService with R, Throwable, T] = " +
Copy link
Collaborator Author

@myazinn myazinn Apr 25, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still compiles just fine. I guess it was a workaround for a bug rather than a feature

@myazinn myazinn force-pushed the 429_propagate_context_to_reneder_param_type_correctly branch from 6196cac to 5e3ba64 Compare April 25, 2023 19:31
@myazinn myazinn merged commit 6db0245 into zio:idea231.x Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@accessible macro highlights as error but runs in tests
1 participant