Skip to content
New issue

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

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

Already on GitHub? # to your account

data-converted: always sort keys in output JSON #2343

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

ekiwi
Copy link
Contributor

@ekiwi ekiwi commented Nov 6, 2024

Let's see if this works

@ekiwi ekiwi marked this pull request as ready for review November 6, 2024 17:00
@ekiwi ekiwi requested a review from rachitnigam November 6, 2024 17:00
@ekiwi
Copy link
Contributor Author

ekiwi commented Nov 6, 2024

@rachitnigam : Let me know if you want to get #2341 in first. In that case, I can rebase this PR. Otherwise, #2341 will have to rebase after we merge this.

@rachitnigam
Copy link
Contributor

LGTM! Let's get #2341 in first and then merge this.

@ekiwi ekiwi force-pushed the data-converted-sorted-keys branch from 221a4e5 to fd4c216 Compare November 6, 2024 17:33
@ekiwi ekiwi force-pushed the data-converted-sorted-keys branch from fd4c216 to a98ea57 Compare November 6, 2024 17:35
@ekiwi ekiwi enabled auto-merge (squash) November 6, 2024 17:35
@ekiwi ekiwi merged commit fd43d16 into main Nov 6, 2024
18 checks passed
@ekiwi ekiwi deleted the data-converted-sorted-keys branch November 6, 2024 17:50
@EclecticGriffin
Copy link
Collaborator

Awesome! Probably should've thought to do this ages ago!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants