Skip to content

Conversation

@affeldt-aist
Copy link
Member

Co-authored-by: @agontard

Motivation for this change

fixes #1804

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md

- [ ] added corresponding documentation in the headers

Reference: How to document

Merge policy

As a rule of thumb:

  • PRs with several commits that make sense individually and that
    all compile are preferentially merged into master.
  • PRs with disorganized commits are very likely to be squash-rebased.
Reminder to reviewers

Co-authored-by: @agontard
@t6s
Copy link
Member

t6s commented Dec 24, 2025

this change looks contradictory to @CohenCyril 's suggestion in the original discussion

@affeldt-aist
Copy link
Member Author

affeldt-aist commented Dec 24, 2025

Yes, the PR was done quickly to check the CI to fuel the discussion. It could be closed now, except that @agontard maybe found an inconsistency in MathComp proper (shouldn't the arguments of ltP be made explicit after all?).

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.

Implicit argument inconsistency for reflect lemmas

2 participants