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

Inject Product super to generated case classes #296

Merged
merged 2 commits into from
Feb 27, 2024
Merged

Conversation

RustedBones
Copy link
Contributor

Scio 0.14 default coders for generated classes are not found by IntelliJ if the IDE does not know the classes have Product super type

Scio 0.14 default coders for generated classes are not found by IntelliJ
if the IDE does not know the classes have Product super type
@RustedBones
Copy link
Contributor Author

I think this should fix spotify/scio#5243

@RustedBones
Copy link
Contributor Author

Tested on scio with spotify/scio#5274

image

}
} yield defs

result.toSeq
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I doubt this ever worked. This was returning a

List("def List(p1: Type1, p2: Type2) = ???")

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, that does not seem helpful...

Copy link
Contributor

@clairemcginty clairemcginty left a comment

Choose a reason for hiding this comment

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

nice!

}
} yield defs

result.toSeq
Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, that does not seem helpful...

@clairemcginty clairemcginty merged commit 1b6513e into main Feb 27, 2024
3 checks passed
@clairemcginty clairemcginty deleted the inject-product branch February 27, 2024 18:17
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.

2 participants