Skip to content

Support for mutual inductives - #1053

Closed
Janno wants to merge 27 commits into
LPCIC:masterfrom
Janno:janno/mutind
Closed

Support for mutual inductives#1053
Janno wants to merge 27 commits into
LPCIC:masterfrom
Janno:janno/mutind

Conversation

@Janno

@Janno Janno commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Janno added 27 commits June 19, 2026 13:47
Those two thing are not the same.
@Janno Janno mentioned this pull request Jun 19, 2026
@gares

gares commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Thanks, I started to review some code but the current structure is hard to review/merge.

I think it would help to have all changes about .ml files and files in elpi/ and tests/ to be part of a first PR.

Then have all changes about derive/eq to be in a followup commit in another PR.

And so on. This way I can merge things incrementally, and add tests on each of these.

Can you squash then separate the changes in this way?

@VojtechStep

Copy link
Copy Markdown
Contributor

Btw is this the same code as SkyLabsAI#2?

@Janno

Janno commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

Btw is this the same code as SkyLabsAI#2?

Yes, this is the same code rebased on master (instead of our own vendored snapshot).

Can you squash then separate the changes in this way?

Yes, I'll try to get that done ASAP.

@Janno

Janno commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #1054 and #1056

@Janno Janno closed this Jun 24, 2026
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