-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
DOC Update About Us page #31519
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
base: main
Are you sure you want to change the base?
DOC Update About Us page #31519
Conversation
I wonder if the CI failure is due to:
(those image warnings are sure annoying to grep through, I am assuming it is because we don't run the examples so they are missing) |
doc/about.rst
Outdated
.. include:: maintainers_emeritus.rst | ||
.. raw:: html |
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.
the error is coming from here, since the link to maintainers_emeritus.rst
is being removed. I rather keep the link though, to have one place to keep that information. The same way for other sections like the communication team section.
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.
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 much nicer but it would be nice to have this list only in one place, so you don't have to update in 2 places when you want to change.
This may be too annoying to implement/maintain, but it may be possible to have the multi-column list and use include
by having special CSS (that does the multi-column thing) and putting the include
inside a container directive.
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.
I did try several ways and ran into some errors.
{csv-table} Emeritus Maintainers
:widths: 20, 20, 20
"Mathieu Blondel", "Joris Van den Bossche", "Matthieu Brucher"
"Lars Buitinck", "David Cournapeau", "Noel Dawe"
"Vincent Dubourg", "Edouard Duchesnay", "Alexander Fabisch"
"Virgile Fritsch", "Satrajit Ghosh", "Angel Soler Gollonet"
"Chris Gorgolewski", "Jaques Grobler", "Yaroslav Halchenko"
"Brian Holt", "Arnaud Joly", "Thouis (Ray) Jones"
"Kyle Kastner", "Manoj Kumar", "Robert Layton"
"Wei Li", "Paolo Losi", "Gilles Louppe"
"Jan Hendrik Metzen", "Vincent Michel", "Jarrod Millman"
"Vlad Niculae", "Alexandre Passos", "Fabian Pedregosa"
"Peter Prettenhofer", "Hanmin Qin", "(Venkat) Raghav Rajagopalan"
"Jacob Schreiber", "杜世橋 Du Shiqiao", "Bertrand Thirion"
"Tom Dupré la Tour", "Jake Vanderplas", "Nelle Varoquaux"
"David Warde-Farley", "Ron Weiss", "Roman Yurchak"
========================== ========================== ==========================
Name Name Name
========================== ========================== ==========================
Mathieu Blondel Joris Van den Bossche Matthieu Brucher
Lars Buitinck David Cournapeau Noel Dawe
Vincent Dubourg Edouard Duchesnay Alexander Fabisch
Virgile Fritsch Satrajit Ghosh Angel Soler Gollonet
Chris Gorgolewski Jaques Grobler Yaroslav Halchenko
Brian Holt Arnaud Joly Thouis (Ray) Jones
Kyle Kastner Manoj Kumar Robert Layton
Wei Li Paolo Losi Gilles Louppe
Jan Hendrik Metzen Vincent Michel Jarrod Millman
Vlad Niculae Alexandre Passos Fabian Pedregosa
Peter Prettenhofer Hanmin Qin (Venkat) Raghav, Rajagopalan
Jacob Schreiber 杜世橋 Du Shiqiao Bertrand Thirion
Tom Dupré la Tour Jake Vanderplas Nelle Varoquaux
David Warde-Farley Ron Weiss Roman Yurchak
========================== ========================== ==========================
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.
Another option is to remove the new HTML table for the emeritus contributors in this PR, and try it in a new PR, so it's not holding this up. I did make a lot of updates in various sections, so breaking up the PR is one way to move forward.
Reference Issues/PRs
References #30826
What does this implement/fix? Explain your changes.
Any other comments?