Skip to content

fix: invalid dotted identifier notation error for sort #8260

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tydeu
Copy link
Member

@tydeu tydeu commented May 8, 2025

This PR clarifies the invalid dotted identifier notation error when the type is a sort.

Co-authored-by @sgraf812.

@tydeu tydeu added changelog-library Library changelog-language Language features, tactics, and metaprograms and removed changelog-library Library labels May 8, 2025
#guard_msgs in
def foo2 (n : Nat) : Nat :=
match PUnit with
| .unit => n
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe also test uses outside of patterns:

def Prop.true := True
#check (.true : Prop)

@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label May 8, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 9c7cb147b91fd1330764de7336c2ad2745e73f52 --onto 836d7b703ac6b6c74892339785c20940e17fcd7f. You can force Mathlib CI using the force-mathlib-ci label. (2025-05-08 10:40:27)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-language Language features, tactics, and metaprograms toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants