[PR #1019] [MERGED] Introduces Bytes class #853

New Issue

adam · 2025-12-30T01:27:12+01:00

adam commented

2025-12-30 01:27:12 +01:00

📋 Pull Request Information

Original PR: https://github.com/apple/pkl/pull/1019
Author: @bioball
Created: 3/11/2025
Status: ✅ Merged
Merged: 6/12/2025
Merged by: @bioball

Base: main ← Head: bytes-stdlib

📝 Commits (5)

f0f12c6 Introduces Bytes library
9b5c6ff Update pkl-core/src/main/java/org/pkl/core/runtime/VmBytes.java
5395586 Fix test
5a40771 Run spotless apply
dff4717 Fix program value renderer for VmBytes

📊 Changes

104 files changed (+2208 additions, -543 deletions)

View changed files

📝 docs/modules/ROOT/partials/component-attributes.adoc (+1 -0)
📝 docs/modules/bindings-specification/pages/binary-encoding.adoc (+18 -0)
📝 docs/modules/language-reference/pages/index.adoc (+42 -0)
📝 pkl-cli/src/main/kotlin/org/pkl/cli/CliEvaluator.kt (+72 -34)
📝 pkl-cli/src/main/kotlin/org/pkl/cli/CliEvaluatorOptions.kt (+2 -2)
📝 pkl-cli/src/test/kotlin/org/pkl/cli/CliEvaluatorTest.kt (+33 -33)
📝 pkl-codegen-java/src/main/kotlin/org/pkl/codegen/java/JavaCodeGenerator.kt (+19 -6)
📝 pkl-codegen-java/src/test/kotlin/org/pkl/codegen/java/JavaCodeGeneratorTest.kt (+4 -0)
📝 pkl-codegen-java/src/test/resources/org/pkl/codegen/java/PropertyTypes.jva (+40 -29)
📝 pkl-codegen-kotlin/src/main/kotlin/org/pkl/codegen/kotlin/KotlinCodeGenerator.kt (+44 -23)
📝 pkl-codegen-kotlin/src/test/kotlin/org/pkl/codegen/kotlin/KotlinCodeGeneratorTest.kt (+24 -2)
📝 pkl-codegen-kotlin/src/test/resources/org/pkl/codegen/kotlin/PropertyTypes.kotlin (+10 -4)
📝 pkl-config-java/src/main/java/org/pkl/config/java/mapper/Conversions.java (+3 -2)
➕ pkl-config-java/src/test/java/org/pkl/config/java/mapper/BytesToByteArrayTest.java (+46 -0)
➕ pkl-config-java/src/test/resources/org/pkl/config/java/mapper/BytesToByteArrayTest.pkl (+2 -0)
📝 pkl-core/src/main/java/org/pkl/core/Evaluator.java (+14 -1)
📝 pkl-core/src/main/java/org/pkl/core/EvaluatorImpl.java (+34 -21)
📝 pkl-core/src/main/java/org/pkl/core/FileOutput.java (+9 -2)
📝 pkl-core/src/main/java/org/pkl/core/FileOutputImpl.java (+13 -1)
📝 pkl-core/src/main/java/org/pkl/core/JsonRenderer.java (+7 -1)

...and 80 more files

📄 Description

This introduces a new Bytes standard library class, for working with binary data.

Follows SPICE-0013

_{🔄 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/1019 **Author:** [@bioball](https://github.com/bioball) **Created:** 3/11/2025 **Status:** ✅ Merged **Merged:** 6/12/2025 **Merged by:** [@bioball](https://github.com/bioball) **Base:** `main` ← **Head:** `bytes-stdlib` --- ### 📝 Commits (5) - [`f0f12c6`](https://github.com/apple/pkl/commit/f0f12c63adf948e0f32d6002c78dfdb786d90910) Introduces `Bytes` library - [`9b5c6ff`](https://github.com/apple/pkl/commit/9b5c6ff891235b27742e05a91e495aa7fb037e6a) Update pkl-core/src/main/java/org/pkl/core/runtime/VmBytes.java - [`5395586`](https://github.com/apple/pkl/commit/5395586cef045ab7f303890ffd828a6dc4deb334) Fix test - [`5a40771`](https://github.com/apple/pkl/commit/5a40771bf7ad473321d309fb2c07c931cbd69798) Run spotless apply - [`dff4717`](https://github.com/apple/pkl/commit/dff47174e3866480e98d2a0df779f691ce075114) Fix program value renderer for VmBytes ### 📊 Changes **104 files changed** (+2208 additions, -543 deletions) <details> <summary>View changed files</summary> 📝 `docs/modules/ROOT/partials/component-attributes.adoc` (+1 -0) 📝 `docs/modules/bindings-specification/pages/binary-encoding.adoc` (+18 -0) 📝 `docs/modules/language-reference/pages/index.adoc` (+42 -0) 📝 `pkl-cli/src/main/kotlin/org/pkl/cli/CliEvaluator.kt` (+72 -34) 📝 `pkl-cli/src/main/kotlin/org/pkl/cli/CliEvaluatorOptions.kt` (+2 -2) 📝 `pkl-cli/src/test/kotlin/org/pkl/cli/CliEvaluatorTest.kt` (+33 -33) 📝 `pkl-codegen-java/src/main/kotlin/org/pkl/codegen/java/JavaCodeGenerator.kt` (+19 -6) 📝 `pkl-codegen-java/src/test/kotlin/org/pkl/codegen/java/JavaCodeGeneratorTest.kt` (+4 -0) 📝 `pkl-codegen-java/src/test/resources/org/pkl/codegen/java/PropertyTypes.jva` (+40 -29) 📝 `pkl-codegen-kotlin/src/main/kotlin/org/pkl/codegen/kotlin/KotlinCodeGenerator.kt` (+44 -23) 📝 `pkl-codegen-kotlin/src/test/kotlin/org/pkl/codegen/kotlin/KotlinCodeGeneratorTest.kt` (+24 -2) 📝 `pkl-codegen-kotlin/src/test/resources/org/pkl/codegen/kotlin/PropertyTypes.kotlin` (+10 -4) 📝 `pkl-config-java/src/main/java/org/pkl/config/java/mapper/Conversions.java` (+3 -2) ➕ `pkl-config-java/src/test/java/org/pkl/config/java/mapper/BytesToByteArrayTest.java` (+46 -0) ➕ `pkl-config-java/src/test/resources/org/pkl/config/java/mapper/BytesToByteArrayTest.pkl` (+2 -0) 📝 `pkl-core/src/main/java/org/pkl/core/Evaluator.java` (+14 -1) 📝 `pkl-core/src/main/java/org/pkl/core/EvaluatorImpl.java` (+34 -21) 📝 `pkl-core/src/main/java/org/pkl/core/FileOutput.java` (+9 -2) 📝 `pkl-core/src/main/java/org/pkl/core/FileOutputImpl.java` (+13 -1) 📝 `pkl-core/src/main/java/org/pkl/core/JsonRenderer.java` (+7 -1) _...and 80 more files_ </details> ### 📄 Description This introduces a new `Bytes` standard library class, for working with binary data. Follows [SPICE-0013](https://github.com/apple/pkl-evolution/pull/14) --- <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:27:12 +01:00

adam closed this issue

2025-12-30 01:27:12 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pkl#853

[PR #1019] [MERGED] Introduces Bytes class #853

📋 Pull Request Information

📝 Commits (5)

📊 Changes

📄 Description

[PR #1019] [MERGED] Introduces `Bytes` class #853