Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix default value #533

Merged
merged 1 commit into from
Oct 15, 2023
Merged

Fix default value #533

merged 1 commit into from
Oct 15, 2023

Conversation

satabin
Copy link
Member

@satabin satabin commented Oct 15, 2023

Fixes the problem that the default is emitted in objects if the first field is not matching.

Now the default constructor is a first class citizen throughout the process. It is emitted only if it is not followed by a value constructor in the stream processor.

@satabin satabin requested a review from a team as a code owner October 15, 2023 10:28
@satabin satabin added bug Something isn't working json labels Oct 15, 2023
Fixes the problem that the default is emitted in objects if the first
field is not matching.
@satabin satabin merged commit 112aaa9 into main Oct 15, 2023
28 checks passed
@satabin satabin deleted the json/jq/fix-epsilon branch October 15, 2023 11:44
@satabin satabin self-assigned this Oct 15, 2023
@satabin satabin added this to the 1.9 milestone Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working json
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant