[PR #1280] [MERGED] Don't force multiline interpolation into a single line #993

New Issue

adam · 2025-12-30T01:28:09+01:00

adam commented

2025-12-30 01:28:09 +01:00

📋 Pull Request Information

Original PR: https://github.com/apple/pkl/pull/1280
Author: @bioball
Created: 10/31/2025
Status: ✅ Merged
Merged: 10/31/2025
Merged by: @bioball

Base: main ← Head: string-interpolation-formatting-fixes

📝 Commits (3)

fe2516c Don't force multiline expressions into a single line in string interpolation
9149e20 Update pkl-formatter/src/main/kotlin/org/pkl/formatter/Builder.kt
1bc5d88 Add PR number to release notes

📊 Changes

5 files changed (+69 additions, -16 deletions)

View changed files

📝 docs/modules/release-notes/pages/0.30.adoc (+1 -1)
📝 pkl-formatter/src/main/kotlin/org/pkl/formatter/Builder.kt (+13 -13)
📝 pkl-formatter/src/test/files/FormatterSnippetTests/input/string-interpolation.pkl (+26 -1)
📝 pkl-formatter/src/test/files/FormatterSnippetTests/output/multi-line-strings.pkl (+3 -1)
📝 pkl-formatter/src/test/files/FormatterSnippetTests/output/string-interpolation.pkl (+26 -0)

📄 Description

Also, fixes an issue where forced single-line formatting would break for if/else and let expressions

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/apple/pkl/pull/1280 **Author:** [@bioball](https://github.com/bioball) **Created:** 10/31/2025 **Status:** ✅ Merged **Merged:** 10/31/2025 **Merged by:** [@bioball](https://github.com/bioball) **Base:** `main` ← **Head:** `string-interpolation-formatting-fixes` --- ### 📝 Commits (3) - [`fe2516c`](https://github.com/apple/pkl/commit/fe2516cfe54dccd14c2d01d36c956ba395901a48) Don't force multiline expressions into a single line in string interpolation - [`9149e20`](https://github.com/apple/pkl/commit/9149e200b0792e384008718387605a2a5f4c3a63) Update pkl-formatter/src/main/kotlin/org/pkl/formatter/Builder.kt - [`1bc5d88`](https://github.com/apple/pkl/commit/1bc5d881c9ee219e3c5753154024e249c9dfc82a) Add PR number to release notes ### 📊 Changes **5 files changed** (+69 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `docs/modules/release-notes/pages/0.30.adoc` (+1 -1) 📝 `pkl-formatter/src/main/kotlin/org/pkl/formatter/Builder.kt` (+13 -13) 📝 `pkl-formatter/src/test/files/FormatterSnippetTests/input/string-interpolation.pkl` (+26 -1) 📝 `pkl-formatter/src/test/files/FormatterSnippetTests/output/multi-line-strings.pkl` (+3 -1) 📝 `pkl-formatter/src/test/files/FormatterSnippetTests/output/string-interpolation.pkl` (+26 -0) </details> ### 📄 Description Also, fixes an issue where forced single-line formatting would break for if/else and let expressions --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

adam added the pull-request label 2025-12-30 01:28:09 +01:00

adam closed this issue

2025-12-30 01:28:09 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pkl#993