Skip to content

[Merged by Bors] - feat(Mathlib/Data/Matrix/Notation): delaborator for !![a, b; c, d] notation#14376

Closed
Komyyy wants to merge 4 commits intomasterfrom
Komyyy/Matrix.Notation
Closed

[Merged by Bors] - feat(Mathlib/Data/Matrix/Notation): delaborator for !![a, b; c, d] notation#14376
Komyyy wants to merge 4 commits intomasterfrom
Komyyy/Matrix.Notation

Conversation

@Komyyy
Copy link
Copy Markdown
Contributor

@Komyyy Komyyy commented Jul 3, 2024


Open in Gitpod

@Komyyy Komyyy added awaiting-review t-meta Tactics, attributes or user commands awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. t-algebra Algebra (groups, rings, fields, etc) labels Jul 3, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 3, 2024

PR summary 0288892d5c

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ delabMatrixNotation

You can run this locally as follows
## summary with just the declaration names:
./scripts/no_lost_declarations.sh short <optional_commit>

## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit>

@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Jul 3, 2024
@Komyyy Komyyy requested a review from kmill July 10, 2024 12:20
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

Nifty! I'm not super thrilled by the implementation matching on the output of a previous delab, but I also don't have a case in mind where it fails.

I'll let Kyle give it a final check.

@Komyyy Komyyy added the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Jul 11, 2024
@Komyyy Komyyy changed the title feat: delaborator for !![a, b; c, d] notation feat(Mathlib/Data/Matrix/Notation): delaborator for !![a, b; c, d] notation Jul 11, 2024
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Jul 11, 2024
@kmill
Copy link
Copy Markdown
Contributor

kmill commented Jul 11, 2024

Thanks!

bors r+

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Jul 11, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jul 11, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 11, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Mathlib/Data/Matrix/Notation): delaborator for !![a, b; c, d] notation [Merged by Bors] - feat(Mathlib/Data/Matrix/Notation): delaborator for !![a, b; c, d] notation Jul 11, 2024
@mathlib-bors mathlib-bors bot closed this Jul 11, 2024
@mathlib-bors mathlib-bors bot deleted the Komyyy/Matrix.Notation branch July 11, 2024 20:32
@adomani adomani mentioned this pull request Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc) t-meta Tactics, attributes or user commands

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants