-
Notifications
You must be signed in to change notification settings - Fork 340
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
[Merged by Bors] - feat: Incidence algebras #8900
Conversation
PR summary e755ff8535Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
Mathlib/Algebra/Order/Incidence.lean
Outdated
/-! -/ | ||
-- fake module docstring just so that I can disable the long line linter without getting linted for | ||
-- a missing module docstring! | ||
|
||
set_option linter.style.longLine false in | ||
/-! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@grunweg, you might want to investigate 😇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question! @damiano You are the expert on this part - with the header linter, what's the best way to silence the linter here? One should certainly be able to do (untested)
set_option linter.style header false in
set_option linter.style.longLine false in
stuff
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be clear, I would be even happier if #8900 (comment) didn't trigger the long line linter at all
Mathlib/Algebra/Order/Incidence.lean
Outdated
|
||
* [Aigner, *Combinatorial Theory, Chapter IV*][aigner1997] | ||
* [Jacobson, *Basic Algebra I, 8.6*][jacobson1989] | ||
* [Doubilet, Rota, Stanley, *On the foundations of Combinatorial Theory VI*][doubilet_rota_stanley_vi] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the line that's too long
Define incidence algebras From LeanCamCombi Co-authored-by: Alex J. Best <alex.j.best@gmail.com>
Co-authored-by: Bhavik Mehta <bhavikmehta8@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors d+
✌️ YaelDillies can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
Define incidence algebras From LeanCamCombi Co-authored-by: Alex J. Best <alex.j.best@gmail.com>
Pull request successfully merged into master. Build succeeded: |
@@ -2035,6 +2067,19 @@ @Book{ Jacobson1956 | |||
zbl = {0073.02002} | |||
} | |||
|
|||
@Book{ jacobson1989, | |||
author = {Jacobson, Nathan}, | |||
title = {Basic algebra. {II}}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be Basic Algebra I?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for spotting! I've opened #19027
Define incidence algebras
From LeanCamCombi
Co-authored-by: Alex J. Best alex.j.best@gmail.com