From cbb94e11dea2e6a94d9e68f6782494d3c36c9c79 Mon Sep 17 00:00:00 2001
From: Chris Markiewicz <markiewicz@stanford.edu>
Date: Thu, 25 Apr 2024 06:41:47 -0400
Subject: [PATCH] [MAINT] Update contributor table generator style (#1805)

* chore(code-style): Rewrite f-strings in print_contributors.py to outrage flake8 less

* chore(render): Rerender contributors.md

* chore(style): Ignore linting errors in auto-generated table
---
 src/appendices/contributors.md | 13 +++++++------
 tools/print_contributors.py    | 20 +++++++++-----------
 2 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/src/appendices/contributors.md b/src/appendices/contributors.md
index 2ffab4740f..f48586f04d 100644
--- a/src/appendices/contributors.md
+++ b/src/appendices/contributors.md
@@ -37,6 +37,7 @@ ecosystem (in alphabetical order).
 If you contributed to the BIDS ecosystem and your name is not listed, please add it.
 
 <!-- THE SECTION BELOW IS AUTOMATICALLY GENERATED -->
+<!--lint ignore -->
 
 | name                                                 | contributions                          |
 | ---------------------------------------------------- | -------------------------------------- |
@@ -121,7 +122,7 @@ If you contributed to the BIDS ecosystem and your name is not listed, please add
 | Dianne Patterson                                     | 📖                                     |
 | Dimitri Papadopoulos Orfanos                         | 📖💡🤔💬💻                             |
 | Dmitry Petrov                                        | 📖💻                                   |
-| Dora Hermes                                          | 📖💻✅🔍🤔                              |
+| Dora Hermes                                          | 📖💻✅🔍🤔                             |
 | Dorien Huijser                                       | 📖                                     |
 | Douglas N. Greve                                     | 📖                                     |
 | Duncan Macleod                                       | 📖🚇                                   |
@@ -194,7 +195,7 @@ If you contributed to the BIDS ecosystem and your name is not listed, please add
 | Jeanette Mumford                                     | 📖                                     |
 | Jefferson Casimir                                    | 🔧                                     |
 | Jeffrey G. Ojemann                                   | 📖                                     |
-| Jeffrey S. Grethe                                    | 💬🐛✅📢💻                              |
+| Jeffrey S. Grethe                                    | 💬🐛✅📢💻                             |
 | JegouA                                               | 💻                                     |
 | Jelle Dalenberg                                      | 📖                                     |
 | Jeremy Moreau                                        | 📖💡                                   |
@@ -211,7 +212,7 @@ If you contributed to the BIDS ecosystem and your name is not listed, please add
 | Jose Manuel Saborit                                  | 📖                                     |
 | Joseph Wexler                                        | 📖💡                                   |
 | Joseph Woods                                         | 📖                                     |
-| Julia Guiomar Niso Galán                             | 🤔🎨🔍👀📋📝🔧🐛💻🔣✅💬📖💡📢          |
+| Julia Guiomar Niso Galán                             | 🤔🎨🔍👀📋📝🔧🐛💻🔣✅💬📖💡📢         |
 | Julia Sprenger                                       | 📖                                     |
 | Julien Cohen-Adad                                    | 📖🔣🤔                                 |
 | Julius Welzel                                        | 📖💡🐛💻🔣🤔💬📓                       |
@@ -269,7 +270,7 @@ If you contributed to the BIDS ecosystem and your name is not listed, please add
 | Michael Hanke                                        | 📖🤔🔧🐛📢                             |
 | Michael P. Harms                                     | 📖⚠️🔧                                 |
 | Michael P. Milham                                    | 💡🔍                                   |
-| Michael P. Notter                                    | 💬📝✅📢📖                              |
+| Michael P. Notter                                    | 💬📝✅📢📖                             |
 | Michael Schirner                                     | 📖                                     |
 | Mikaël Naveau                                        | 🐛                                     |
 | Nader Pouratian                                      | 📖                                     |
@@ -288,7 +289,7 @@ If you contributed to the BIDS ecosystem and your name is not listed, please add
 | Patricia Clement                                     | 💬🐛💻📖🔣💡📋🤔📆⚠️📢                 |
 | Patrick Park                                         | 📖💡💬💻                               |
 | Paule-Joanne Toussaint                               | 📖                                     |
-| Peer Herholz                                         | 💬📖👀🔧✅📢                            |
+| Peer Herholz                                         | 💬📖👀🔧✅📢                           |
 | Petra Ritter                                         | 📖                                     |
 | Pierre Rioux                                         | 📖                                     |
 | Pieter Vandemaele                                    | 📖💻                                   |
@@ -318,7 +319,7 @@ If you contributed to the BIDS ecosystem and your name is not listed, please add
 | Shashank Bansal                                      | 📖                                     |
 | Sjoerd B. Vos                                        | 📖                                     |
 | Soichi Hayashi                                       | 📖🔧🐛                                 |
-| Stefan Appelhoff                                     | 📖💬🤔🐛💡💻👀⚠️📢✅🔧🔌📝🚧🔣          |
+| Stefan Appelhoff                                     | 📖💬🤔🐛💡💻👀⚠️📢✅🔧🔌📝🚧🔣         |
 | Stephan Bickel                                       | 📖                                     |
 | Steven Meisler                                       | 🐛💻💬🔧📓                             |
 | Suyash Bhogawar                                      | 📖💡⚠️🔧💬                             |
diff --git a/tools/print_contributors.py b/tools/print_contributors.py
index c49ca6f823..29db86a39c 100644
--- a/tools/print_contributors.py
+++ b/tools/print_contributors.py
@@ -15,8 +15,9 @@
 
 
 def contributor_table_header(max_name_length, max_contrib_length):
-    return f"""| name{" " * (max_name_length-4)} | contributions{" " * (max_contrib_length-13)} |
-| {"-" * max_name_length} | {"-"*max_contrib_length} |
+    return f"""\
+| {"name":<{max_name_length}} | {"contributions":<{max_contrib_length}} |
+| {"":-<{max_name_length}} | {"":-<{max_contrib_length}} |
 """
 
 
@@ -24,16 +25,13 @@ def create_line_contributor(
     contributor: dict[str, str], max_name_length: int, max_contrib_length: int
 ):
     name = contributor["name"]
+    emap = emoji_map()
+    contributions = "".join(
+        emoji.emojize(emap[cont]) for cont in contributor["contributions"]
+    )
 
-    line = f"| {name}{' '*(max_name_length-len(name))} | "
-
-    nb_contrib = len(contributor["contributions"]) * 2
-    for contrib in contributor["contributions"]:
-        line += emoji.emojize(emoji_map()[contrib])
-
-    line += f"{' '*(max_contrib_length-nb_contrib)} |\n"
-
-    return line
+    pad = max_contrib_length - len(contributor["contributions"]) * 2
+    return f"| {name:<{max_name_length}} | {contributions}{'':<{pad}} |\n"
 
 
 def main():