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.

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

refactor: ArgsPacker.unpack to return Option #6359

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

nomeata
Copy link
Collaborator

@nomeata nomeata commented Dec 10, 2024

so that it can be used in pure code and that the error message can be
adjusted

so that it can be used in pure code and that the error message can be
adjusted
@nomeata nomeata added the will-merge-soon …unless someone speaks up label Dec 10, 2024
@nomeata nomeata enabled auto-merge December 10, 2024 14:37
@nomeata nomeata requested a review from leodemoura as a code owner December 10, 2024 15:07
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc December 10, 2024 15:19 Inactive
@nomeata nomeata added this pull request to the merge queue Dec 10, 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 Dec 10, 2024
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 10, 2024
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Dec 10, 2024
Merged via the queue into master with commit d27c5af Dec 10, 2024
16 checks passed
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Dec 10, 2024
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builds-mathlib CI has verified that Mathlib builds against this PR toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN will-merge-soon …unless someone speaks up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants