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 on a pull request on apple/pkl-pantry
Looks good to me! This will need final approval and merge from the core team.

View on GitHub

HT154 created a review comment on a pull request on apple/pkl-pantry
The type constrain here should match the one for `name`

View on GitHub

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

View on GitHub

HT154 created a comment on a pull request on apple/pkl-pantry
Don't forget to bump the package patch version in PklProject. If there are local packages that depend on this one you will need to bump those versions as well and then run `./gradlew resolveProjects`.

View on GitHub

HT154 created a review comment on a pull request on apple/pkl-pantry
I think this condition may be reversed. As-is, it doesn't forbid these properties from being non-null when the type is not `apiKey`. ```suggestion name: String?((this != null).implies(type == "api...

View on GitHub

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

View on GitHub

HT154 created a comment on an issue on apple/pkl-pantry
This is definitely an issue. This repo is open for contribution, so don't hesitate to open a PR for a fix! My reading of the spec is that both `name` and `in` are required if and only if `type == ...

View on GitHub

HT154 created a comment on an issue on apple/pkl
You would need to either a) define a class with a property `default` or use `Mapping` with key `"default"`. You're (implicitly) using `Dynamic` here, which has a property `default` that's marked `h...

View on GitHub

HT154 pushed 10 commits to main HT154/pkl
  • codegen-java: Support not annotating constructor parameters (#792) Motivation: Spring Boot configuration classes ne... 01bf844
  • Convert `org.pkl.executor.Version.Identifier` POJO to `record` class (#836) 6bd8e28
  • Download JDK for windows build (#851) Don't use the system Java on Windows builds, instead download them from Adopti... 9982511
  • Fix exception handling of PklRootNode's (#837) Motivation: - Perform same exception handling for every implementati... 17f4313
  • Fix CreateEvaluatorRequest decoding (#853) Handle case when request specifies external reader with null arguments 6cab470
  • Fix compile error (#857) PRs https://github.com/apple/pkl/pull/789 and https://github.com/apple/pkl/pull/837 being m... efe1608
  • Update JDK version in dev docs (#848) 29fb99f
  • Clarify terminology on renderer converters (#860) 6fd82a5
  • Add comment about const check (#859) a014e8d
  • Fix spelling errors (#846) Also: change checksums due to filename and content changes, and fix language snippet test... d9c65d4

View on GitHub

HT154 pushed 2 commits to main HT154/pkl-pantry
  • [com.influxdata.telegraf] Add Jolokia2 input plugin (#89) 267ad65
  • [pkl.experimental.deepToTyped] Add support for key transformations during apply (#88) 90b75bf

View on GitHub

HT154 created a comment on an issue on apple/pkl
I'm not able to reproduce this on Pkl 0.27.1 (the current release). Given module: ```pkl local function typeRegex(name: String): Regex = Regex("./src/([A-Za-z]+)/" + name + ".pkl") // ...

View on GitHub

HT154 deleted a branch HT154/pkl

external-readers-pkl-swift-fix-main

HT154 deleted a branch HT154/pkl

external-readers-pkl-swift-fix

HT154 pushed 1 commit to external-readers HT154/pkl-swift
  • Fix CreateEvaluatorRequest decoding when request specifies external reader with null arguments b4a209e

View on GitHub

HT154 created a branch on HT154/pkl

external-readers-pkl-swift-fix - A configuration as code language with rich validation and tooling.

HT154 deleted a branch HT154/pkl-go

fix-fib

HT154 created a comment on a pull request on apple/pkl
Try installing/selecting a JDK 17 version. I'm not sure of the exact version range that works, but I know that Pkl won't build on JDK 22+.

View on GitHub

HT154 opened a pull request on apple/pkl-go
Fix external reader test fibonacci implementation
Realized this looked funny while working on https://github.com/apple/pkl-swift/pull/26. This fixes the off-by-one error.
HT154 pushed 1 commit to fix-fib HT154/pkl-go
  • Fix external reader test fibonacci implementation 63c4c82

View on GitHub

HT154 created a review comment on a pull request on apple/pkl-swift
File under: Swift behavior that caught me way off guard

View on GitHub

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

View on GitHub

HT154 pushed 7 commits to external-readers HT154/pkl-swift
  • Fix temporary directory creation to avoid failing pkl-core tests (#28) 1d0bf43
  • Make PathElement initializer explicitly public (#31) Also, fix doc comments 7d451c5
  • Implement SPICE-0009 External Readers [SPICE-0009](https://github.com/apple/pkl-evolution/pull/10) 818fc37
  • New close flow b780f05
  • docs 294fe9f
  • Update message type codes 3d77b18
  • Add external reader e2e test a1763f4

View on GitHub

HT154 created a review comment on a pull request on apple/pkl-go
Huh. VSCode yelled at me when I first tried this, but I think I just needed to kick gopls for it to figure things out. Neat!

View on GitHub

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

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl-go
  • Fix deadlocks when running external readers under pkl-go evaluators 39c9b95

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl-go
  • Fix deadlocks when running external readers under pkl-go evaluators 2def5f9

View on GitHub

HT154 pushed 1 commit to external-readers HT154/pkl-go
  • Fix deadlocks when running external readers under pkl-go evaluators 6d3d6b6

View on GitHub

HT154 created a review comment on a pull request on apple/pkl-go
The `unix` build tag doesn't exist, these must be valid `GOOS` values. I could change this to `linux || darwin`, but I think `!windows` is actually more correct here and doesn't rule out [various o...

View on GitHub

Load more