Skip to content

Support for labeled tuples - #80

Merged
pedrobslisboa merged 5 commits into
mainfrom
lab-tuples
Jul 6, 2026
Merged

Support for labeled tuples#80
pedrobslisboa merged 5 commits into
mainfrom
lab-tuples

Conversation

@andreypopp

Copy link
Copy Markdown
Collaborator

No description provided.

@andreypopp

andreypopp commented Apr 23, 2026

Copy link
Copy Markdown
Collaborator Author

@anmonteiro I've bumped ppxlib but nix insists on older 0.37 version... what did I do wrong?

@Khady

Khady commented May 12, 2026

Copy link
Copy Markdown
Collaborator

probably worth adding a test for a tuple that doesn't have all of its members labelled? Like (x:int * string)

@pedrobslisboa

Copy link
Copy Markdown
Collaborator

@andreypopp I pushed a commit into this branch to support the partially labeled

@pedrobslisboa pedrobslisboa mentioned this pull request Jul 1, 2026
@pedrobslisboa

Copy link
Copy Markdown
Collaborator

@andreypopp, with the partially labeled, we have the json on non-label as the index of the tuple. Like this:

type t = (x:int * string)[@@deriving json]
{
  "x": 100,
  "1": "Foo"
}

@pedrobslisboa
pedrobslisboa force-pushed the lab-tuples branch 3 times, most recently from 8cf7fb3 to 5c16427 Compare July 6, 2026 15:38
@pedrobslisboa
pedrobslisboa changed the base branch from main to format July 6, 2026 16:50
@pedrobslisboa
pedrobslisboa force-pushed the format branch 2 times, most recently from ef5b55f to 8fc856f Compare July 6, 2026 17:14
Base automatically changed from format to main July 6, 2026 17:27
@pedrobslisboa
pedrobslisboa merged commit 26477d5 into main Jul 6, 2026
9 checks passed
@pedrobslisboa
pedrobslisboa deleted the lab-tuples branch July 6, 2026 17:44
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.

3 participants