-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
|
Your choice if this PR supports multiline strings or not. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good progress!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good progress! Some nitpick comments, looks like you'll need to at least mock-out LiteralValue
so that the tests can run.
This PR isn't about correctly implementing
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent progress! This was a hard one, and you got the hardest parts working!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work! I think this is ready merge. @SDelgado-21 if you agree click "Ready for review" and then "Merge pull request".
I would set the commit message to "Ported SourceFile (#20)" but it's not crucial.
Implementation: https://github.com/diffplug/selfie-python-wip/blob/main/jvm/selfie-lib/src/commonMain/kotlin/com/diffplug/selfie/guts/SourceFile.kt
Tests: https://github.com/diffplug/selfie-python-wip/blob/main/jvm/selfie-lib/src/commonTest/kotlin/com/diffplug/selfie/guts/SourceFileToBeTest.kt