Creating SourceFile#20
Conversation
|
|
Your choice if this PR supports multiline strings or not. |
nedtwigg
left a comment
There was a problem hiding this comment.
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
|
nedtwigg
left a comment
There was a problem hiding this comment.
Excellent progress! This was a hard one, and you got the hardest parts working!
nedtwigg
left a comment
There was a problem hiding this comment.
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