-
Notifications
You must be signed in to change notification settings - Fork 3
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
Method references supported in transform and fromExpression #133
Method references supported in transform and fromExpression #133
Conversation
b3c1489
to
e3a3b00
Compare
Sonar is a false positive (secrets cannot be used in forked PR's...) I'm gonna disable SonarQube |
Can you also add this change to the changelog in |
Yes, done |
Sorry, I just did the same 😓 I also removed Sonar in branches, so the next build after a merge with main should succeed. |
Fixed the conflicts 😄 |
compiler-plugin/src/main/kotlin/tech/mappie/resolving/classes/ExplicitClassMappingCollector.kt
Outdated
Show resolved
Hide resolved
compiler-plugin/src/main/kotlin/tech/mappie/resolving/classes/ExplicitClassMappingCollector.kt
Outdated
Show resolved
Hide resolved
compiler-plugin/src/test/kotlin/tech/mappie/testing/objects/MethodReferenceTest.kt
Outdated
Show resolved
Hide resolved
compiler-plugin/src/test/kotlin/tech/mappie/testing/objects/MethodReferenceTest.kt
Outdated
Show resolved
Hide resolved
compiler-plugin/src/test/kotlin/tech/mappie/testing/objects/MethodReferenceTest.kt
Outdated
Show resolved
Hide resolved
…into allowMethodReferences
LGTM. Thanks for the PR! |
No description provided.