Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore: restrict exists_prop' to Type #5319

Closed
wants to merge 2 commits into from
Closed

chore: restrict exists_prop' to Type #5319

wants to merge 2 commits into from

Conversation

kim-em
Copy link
Collaborator

@kim-em kim-em commented Sep 12, 2024

No description provided.

@kim-em kim-em enabled auto-merge September 12, 2024 04:53
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc September 12, 2024 05:04 Inactive
@kim-em kim-em added this pull request to the merge queue Sep 12, 2024
@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 Sep 12, 2024
@leanprover-community-mathlib4-bot
Copy link
Collaborator

leanprover-community-mathlib4-bot commented Sep 12, 2024 •
edited by leanprover-community-bot
Loading

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase adfd6c090ef7d6bdcc8f69c5dd6f919b1b71cab3 --onto 27bf7367caf2df7fb44cfd9a1abed556d14dfde0. (2024-09-12 05:11:46)
  • 💥 Mathlib branch lean-pr-testing-5319 build failed against this PR. (2024-09-12 05:40:46) View Log
  • 💥 Mathlib branch lean-pr-testing-5319 build failed against this PR. (2024-09-12 06:33:45) View Log

@kim-em kim-em removed this pull request from the merge queue due to a manual request Sep 12, 2024
@kim-em kim-em force-pushed the restrict_exists_prop branch from bae7d58 to 28df84e Compare September 12, 2024 05:22
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Sep 12, 2024
@leanprover-community-mathlib4-bot
Update lean-toolchain for testing leanprover/lean4#5319
34edd67
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Sep 12, 2024
@leanprover-community-mathlib4-bot
Update lean-toolchain for testing leanprover/lean4#5319
be96b4d
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc September 12, 2024 05:40 Inactive
@leanprover-community-bot leanprover-community-bot added the breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan label Sep 12, 2024
kim-em added a commit to leanprover-community/mathlib4 that referenced this pull request Sep 12, 2024
@kim-em
fixes for leanprover/lean4#5319
1341208
@kim-em
Copy link
Collaborator Author

kim-em commented Sep 12, 2024

Sadly, Mathlib demonstrates this is a bad idea.

@kim-em kim-em closed this Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kim-em @leanprover-community-mathlib4-bot @leanprover-community-bot