Skip to content

Conversation

ibgreen-openai
Copy link
Collaborator

Closes #

Background

  • Prep diff for improved JSON playground.

Change List

  • Improve typing of JSON module.

@coveralls
Copy link

coveralls commented May 21, 2025

Coverage Status

coverage: 91.672% (-0.004%) from 91.676%
when pulling 58d7eee on ib/json-cleanup
into d57aa05 on master.

@ibgreen ibgreen marked this pull request as ready for review May 26, 2025 11:57
Copy link
Collaborator

@chrisgervang chrisgervang left a comment

Choose a reason for hiding this comment

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

Nice cleanups and typings. Left a small comment/question.

What is the perf increase coming from? By removing validate? Edit: I misread "prep" for "perf"


merge(configuration) {
for (const key in configuration) {
switch (key) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is the switch necessary?

@ibgreen ibgreen added this to the v9.2 milestone Jul 13, 2025
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.

4 participants