Skip to content

Conversation

@dnezam
Copy link
Contributor

@dnezam dnezam commented Oct 31, 2025

It feels like I am committing a crime just opening a PR of this size, but here we are. Maybe it makes sense to ask ourselves which files we care about and restrict changes to those?

To be clear, I don't have a problem making these changes; it's just that I don't think anyone can claim to have reviewed this beyond glancing over the diffs.

It claimed that it cannot be parsed in modern syntax, but apparently it can.
@dnezam
Copy link
Contributor Author

dnezam commented Oct 31, 2025

bin/build -t3 works (b91bd71, before I removed the comment)

@binghe
Copy link
Member

binghe commented Nov 1, 2025

The only good thing here (for me, and perhaps for others) is that there's no much room left for such grammar changes, i.e. the peace is finally coming.

@mn200
Copy link
Member

mn200 commented Nov 3, 2025

Our long national nightmare is finally over? :)

@mn200 mn200 merged commit a902f2d into HOL-Theorem-Prover:develop Nov 3, 2025
4 checks passed
@dnezam
Copy link
Contributor Author

dnezam commented Nov 3, 2025

I think we are still a few (tens of) thousands of lines away from that 🫠

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