Skip to content

Kotlin: format string literals like the Java annotaton extractor#11296

Merged
smowton merged 1 commit into
github:mainfrom
smowton:smowton/fix/kotlin-string-literal-formatting
Nov 17, 2022
Merged

Kotlin: format string literals like the Java annotaton extractor#11296
smowton merged 1 commit into
github:mainfrom
smowton:smowton/fix/kotlin-string-literal-formatting

Conversation

@smowton
Copy link
Copy Markdown
Contributor

@smowton smowton commented Nov 16, 2022

Java's regular strings are formatted as they appear in source, but we don't easily have this information available in Kotlin. During annotation extraction however it guesses a source rendering because the source is not necessarily available. By formatting to match the annotation extractor, we prepare to ensure consistency with a Java database when extracting annotations as seen by Kotlin.

@smowton smowton requested review from a team as code owners November 16, 2022 12:37
aschackmull
aschackmull previously approved these changes Nov 16, 2022
Copy link
Copy Markdown
Contributor

@aschackmull aschackmull left a comment

Choose a reason for hiding this comment

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

Looks plausible to me.

tamasvajk
tamasvajk previously approved these changes Nov 16, 2022
Java's regular strings are formatted as they appear in source, but we don't easily have this information available in Kotlin. During annotation extraction however it guesses a source rendering because the source is not necessarily available. By formatting to match the annotation extractor, we prepare to ensure consistency with a Java database
when extracting annotations as seen by Kotlin.
@smowton smowton dismissed stale reviews from tamasvajk and aschackmull via cf34dbd November 17, 2022 17:29
@smowton smowton force-pushed the smowton/fix/kotlin-string-literal-formatting branch from 5adb214 to cf34dbd Compare November 17, 2022 17:29
@github-actions github-actions Bot added the Java label Nov 17, 2022
@smowton
Copy link
Copy Markdown
Contributor Author

smowton commented Nov 17, 2022

@cklin done in the corresponding PR

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants