Skip to content

json-sort: init - #501

Open
drupol wants to merge 1 commit into
numtide:mainfrom
drupol:push-oktlntylxwvq
Open

json-sort: init#501
drupol wants to merge 1 commit into
numtide:mainfrom
drupol:push-oktlntylxwvq

Conversation

@drupol

@drupol drupol commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

JSON-Sort: https://crates.io/crates/json-sort

A CLI tool that sorts JSON object keys to make comparisons and diffs easier, without reordering arrays.

@drupol
drupol force-pushed the push-oktlntylxwvq branch from a64207e to fad169d Compare April 24, 2026 08:10
@drupol
drupol marked this pull request as ready for review April 24, 2026 08:11
@drupol

drupol commented May 20, 2026

Copy link
Copy Markdown
Contributor Author

Friendly ping @brianmcgee

@jfly jfly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, just one bit of confusion inline.

Comment thread programs/json-sort.nix Outdated
@drupol
drupol force-pushed the push-oktlntylxwvq branch from fad169d to f17c31e Compare May 20, 2026 18:02
Comment thread programs/json-sort.nix Outdated
@drupol
drupol force-pushed the push-oktlntylxwvq branch from f17c31e to 5fcfea3 Compare May 20, 2026 18:10
@drupol

drupol commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

@jfly Anything else I can do here ?

@jfly jfly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry, I hadn't seen that you updated the PR. LGTM!

@drupol

drupol commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

@zimbatm @jfly Is there anything left in here ? I really wish to see this land :)

@jfly

jfly commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Still LGTM, but I don't have the commit bit here. @brianmcgee?

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