Skip to content

Fix #514. Fix multiline inline#527

Merged
lieryan merged 3 commits intomasterfrom
lieryan-514-fix-multiline-inline
Nov 21, 2022
Merged

Fix #514. Fix multiline inline#527
lieryan merged 3 commits intomasterfrom
lieryan-514-fix-multiline-inline

Conversation

@lieryan
Copy link
Member

@lieryan lieryan commented Nov 21, 2022

Description

Retain multiline expressions when inlining. Previously, we'd try to join the lines, which is rarely desirable. The new behavior preserves original code's formatting as much as possible.

Fixes #514

Checklist (delete if not relevant):

  • I have added tests that prove my fix is effective or that my feature works
  • I have updated CHANGELOG.md

@lieryan lieryan self-assigned this Nov 21, 2022
@lieryan lieryan added this to the 1.5.0 milestone Nov 21, 2022
@lieryan lieryan enabled auto-merge November 21, 2022 17:29
@lieryan lieryan merged commit a261b13 into master Nov 21, 2022
@lieryan lieryan deleted the lieryan-514-fix-multiline-inline branch November 21, 2022 17:31
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.

Fail to inline dictionary with inline comment

1 participant