Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

HT154

HT154 created a review comment on a pull request on apple/pkl
I pushed a workaround that adds a constructor for `ReadResourceResponse` without `content`, but I'm really not in love with it.

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl
  • Work around weird kt interop issue fee8528

View on GitHub

HT154 created a review comment on a pull request on apple/pkl
Pushed this change, but I'm still getting build errors locally from what seems like the same issue. Everything checks out according to IntelliJ but compilation fails with ``` > Task :pkl-server:c...

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl

View on GitHub

HT154 created a review comment on a pull request on apple/pkl
Ah thank you! I struggled to find any hints toward this when I searched. Will get this updated shortly!

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 created a comment on an issue on apple/pkl
This is currently expected behavior and not a bug, but this is definitely a valid feature request. You can emulate this using `-x 'module.renderer.renderValue(<expr>)`. You can see how the outpu...

View on GitHub

HT154 pushed 10 commits to parse-dynamic-disallow-default HT154/pkl
  • Produce more helpful output when module output is overwritten (#716) Fix a bug where overwriting `output` causes a P... 4b6bc7b
  • Fix lazy type checking of UInt types (#740) For example, Listing<UInt16> is equivalent to Listing<UInt16>, but not t... 1ceb489
  • Report error on circular local dependencies (#731) If a stack overflow is found during project evaluation, present a... 93cc325
  • Store `PklProject` annotations in the project metadata (#708) Write annotations to project metadata, and provide the... 8b0118f
  • codegen-kotlin: Support Java serialization of module classes (#721) Motivation: - Java serialization makes as much ... 8fa3acf
  • codegen-java/kotlin: Support Spring Boot 3.x instead of 2.x (#729) Motivation: In Spring Boot 3.0, the annotation t... 08be6be
  • Exclude non-ascii files from windows native snippet tests (#739) There is an existing bug that is preventing the CLI... 466ae6f
  • Error when JSON/YAML parsing would set property `default` in a Dynamic Resolves #561 1478941
  • Improved error messages 187f6bb
  • Apply suggestions from code review Co-authored-by: Philip K.F. Hölzenspies <[email protected]> 13e3306

View on GitHub

HT154 pushed 21 commits to external-readers HT154/pkl
  • Produce more helpful output when module output is overwritten (#716) Fix a bug where overwriting `output` causes a P... 4b6bc7b
  • Fix lazy type checking of UInt types (#740) For example, Listing<UInt16> is equivalent to Listing<UInt16>, but not t... 1ceb489
  • Report error on circular local dependencies (#731) If a stack overflow is found during project evaluation, present a... 93cc325
  • Store `PklProject` annotations in the project metadata (#708) Write annotations to project metadata, and provide the... 8b0118f
  • codegen-kotlin: Support Java serialization of module classes (#721) Motivation: - Java serialization makes as much ... 8fa3acf
  • codegen-java/kotlin: Support Spring Boot 3.x instead of 2.x (#729) Motivation: In Spring Boot 3.0, the annotation t... 08be6be
  • Exclude non-ascii files from windows native snippet tests (#739) There is an existing bug that is preventing the CLI... 466ae6f
  • Refactor message-passing logic into pkl-core This is preparatory work for [SPICE-0009](https://github.com/apple/pkl-... 426237a
  • Split server-specific messages/encoding/decoding back into pkl-server dd29012
  • Implement SPICE-0009 External Readers [SPICE-0009](https://github.com/apple/pkl-evolution/pull/10) New close flow d27908b
  • docs 472b89d
  • Add ExternalReaderRuntime, lots of test coverage 42ce6f0
  • Add doc comments to new public APIs 4f5a72d
  • Improve backcompat for new language binding libs on old pkl versions Box Bytes, fix error CLI handling during projec... b50f801
  • Apply suggestions from code review Co-authored-by: Daniel Chao <[email protected]> Apply suggestions from cod... b5b6a59
  • Address review feedback ff3f1e0
  • Use records for message types d772e5c
  • Update message type codes 13cee73
  • Update naming for resource readers, module key + factory; Move external resolvers into module/resource packages dae7394
  • integrate changes from rebase 2efb4c2
  • and 1 more ...

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl

View on GitHub

HT154 created a review comment on a pull request on apple/pkl
Yeah, this appears to not play nice when bridging into Kotlin: <img width="1862" alt="Screenshot 2024-10-24 at 11 51 25 AM" src="https://github.com/user-attachments/assets/b1516c1d-b96c-4c15-b8ce-...

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 created a review comment on a pull request on apple/pkl
Yup, these appear to be legit: <img width="695" alt="Screenshot 2024-10-24 at 11 20 45 AM" src="https://github.com/user-attachments/assets/635c37a7-134e-4d06-a9ad-b55f03c8acff"> ``` > Task :pk...

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 deleted a branch HT154/pkl

handle-overridden-module-output

HT154 created a review comment on a pull request on apple/pkl
I don't think it's msgpack-specific. This is what I'm seeing: <img width="1740" alt="Screenshot 2024-10-24 at 9 36 27 AM" src="https://github.com/user-attachments/assets/5d188895-159f-41ca-975a-...

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 pushed 17 commits to external-readers HT154/pkl
  • Fix test failures on main (#735) 069a5d1
  • Exclude thrown ascii filename error from native tests (#737) The URI encoding is not predictable here. 22c9a6c
  • codegen-java: Make stateless classes instantiable (#734) Motivation: Currently, the condition for making a generate... cbbcca0
  • Refactor message-passing logic into pkl-core This is preparatory work for [SPICE-0009](https://github.com/apple/pkl-... 474d093
  • Split server-specific messages/encoding/decoding back into pkl-server fb03255
  • Implement SPICE-0009 External Readers [SPICE-0009](https://github.com/apple/pkl-evolution/pull/10) New close flow 545215b
  • docs 8c15817
  • Add ExternalReaderRuntime, lots of test coverage 5ed8c6b
  • Add doc comments to new public APIs 8b7969e
  • Improve backcompat for new language binding libs on old pkl versions Box Bytes, fix error CLI handling during projec... 264ba31
  • Apply suggestions from code review Co-authored-by: Daniel Chao <[email protected]> Apply suggestions from cod... 672de48
  • Address review feedback 397e665
  • Use records for message types 28b2148
  • Update message type codes 4c54645
  • Update naming for resource readers, module key + factory; Move external resolvers into module/resource packages 25eeafa
  • integrate changes from rebase 5cfefb7
  • Update Strings.kt Co-authored-by: Islon Scherer <[email protected]> 8b3ecdd

View on GitHub

HT154 pushed 6 commits to parse-dynamic-disallow-default HT154/pkl
  • Fix test failures on main (#735) 069a5d1
  • Exclude thrown ascii filename error from native tests (#737) The URI encoding is not predictable here. 22c9a6c
  • codegen-java: Make stateless classes instantiable (#734) Motivation: Currently, the condition for making a generate... cbbcca0
  • Error when JSON/YAML parsing would set property `default` in a Dynamic Resolves #561 db1c43a
  • Improved error messages ad638b8
  • Apply suggestions from code review Co-authored-by: Philip K.F. Hölzenspies <[email protected]> f5f96df

View on GitHub

HT154 pushed 6 commits to handle-overridden-module-output HT154/pkl
  • Fix test failures on main (#735) 069a5d1
  • Exclude thrown ascii filename error from native tests (#737) The URI encoding is not predictable here. 22c9a6c
  • codegen-java: Make stateless classes instantiable (#734) Motivation: Currently, the condition for making a generate... cbbcca0
  • Produce more helpful output when module output is overwritten 49b1f2b
  • Update EvaluatorImpl.java Co-authored-by: translatenix <[email protected]> 91e7d8f
  • Move snippet tests to errors, avoid reusing moduleOutputValueTypeMismatch 07bdf17

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl

View on GitHub

HT154 created a review comment on a pull request on apple/pkl
Maybe this is a tooling issue instead, but I wasn't able to get things working correctly with marking `byte[]` usage `@Nullable`. Some of the message passing APIs don't work correctly at all if you...

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 created a review comment on a pull request on apple/pkl
I though about this, but figured this approach (which matches the original Kotlin implementation) might have been chosen for efficiency reasons. This code is in a pretty hot path, so if the goal is...

View on GitHub

HT154 created a review on a pull request on apple/pkl

View on GitHub

HT154 deleted a branch HT154/pkl

fix-pkl-grade-tests-test

HT154 pushed 27 commits to external-readers HT154/pkl
  • codegen-kotlin: Fix generation of copy() methods (#705) - don't generate copy methods for abstract classes - precis... 84f4ec8
  • List Pkl GitHub repositories in README (#715) 9abf1de
  • Fix build performance issues of spotless tasks (#722) Using more efficient glob expressions and removing `targetExcl... 5057bb5
  • Add support for const object members (#678) This adds support for adding the `const` modifier to object members. ... 0ee3d37
  • Remove `isMappingDefault` on `VmNull` (#724) This method is unused. a7cc098
  • Add version of resolved Pkl distribution to `ExecutorException` (#719) * Add version of resolved Pkl distribution to... 0aa4819
  • Change license header from doc- to block-comment (#730) * Change license header from doc to block comment * Apply... eb3891b
  • Add analyze imports libs (SPICE-0001) (#695) This adds a new feature to build a dependency graph of Pkl programs, fo... ce25cb8
  • Fix typecheck error on Listing/Mapping (#725) The addresses the following case: local l1: Listing<String> = n... 9d10832
  • Fix formatting issues (#733) 5a654e4
  • Fix handling of `file:` module URIs with non-ASCII characters (#696) Addresses an issue where Pkl cannot evaluate fi... f9fe226
  • codegen-kotlin: Don't generate equals/hashCode/toString/copy/:Serializable for abstract classes (#714) * codegen-kot... 7302578
  • codegen-java: Change condition for generating equals/hashCode/toString/with/Serializable (#710) Motivation: - Curre... 2040f14
  • Improve testing with stats and errors per test section (#498) * Emojis are moves to the left to be aligned * A summ... 86d870b
  • Refactor message-passing logic into pkl-core This is preparatory work for [SPICE-0009](https://github.com/apple/pkl-... 8e91056
  • Split server-specific messages/encoding/decoding back into pkl-server 28ff6e9
  • Implement SPICE-0009 External Readers [SPICE-0009](https://github.com/apple/pkl-evolution/pull/10) New close flow 3932cb7
  • docs aa9d8b1
  • Add ExternalReaderRuntime, lots of test coverage fa3bd6a
  • Add doc comments to new public APIs 7ce42df
  • and 7 more ...

View on GitHub

Load more