Skip to content

Add starlark output format#1445

Draft
fmeum wants to merge 1 commit intoapple:mainfrom
fmeum:starlark
Draft

Add starlark output format#1445
fmeum wants to merge 1 commit intoapple:mainfrom
fmeum:starlark

Conversation

@fmeum
Copy link
Copy Markdown

@fmeum fmeum commented Feb 26, 2026

TODO:

  • Add support for Pair (tuple)
  • Insert empty lines around top-level statements
  • Sort keyword arguments
  • Find a way to annotate classes with a Bazel load label, collect the labels for all classes in the module and emit them at the top

Copy link
Copy Markdown
Contributor

@HT154 HT154 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi there! We generally encourage renderer development to be done in Pkl code to avoid increasing the API surface and maintenance cost of the core language. Pkl code may subclass ValueRenderer and directly implement its methods. This is how the TOML renderer is implemented: https://github.com/apple/pkl-pantry/tree/main/packages/pkl.toml. It would be much easier for us to accept a contribution of that nature.

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.

2 participants