Skip to content

Add EcoLogits contributors to team section - #28

Merged
inimaz merged 8 commits into
mainfrom
feature/ecologits-contributors
Sep 28, 2026
Merged

inimaz merged 8 commits into
mainfrom
feature/ecologits-contributors

Conversation

@davidberenstein1957

@davidberenstein1957 davidberenstein1957 commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds GitHub links for the EcoLogits contributors and confirms the existing contributor entries, addressing @inimaz's review. Also rewrites the Team section layout and touches copy on the hero and a blog post/ecosystem tagline.

  • Added github_url for Benoît Courty, Amine Saboni, Sasha Luccioni, Iñigo Imaz, Luis Blanche, Patrick and Claire Saignol. Samuel Rincé, Adrien Banse, Vincent Duarte and David Berenstein already had one. Valentin Defour is the only contributor without a link (see below).
  • Restored Sasha Luccioni's organisation: "Hugging Face" and the "adding a contributor" guide comment that were dropped earlier in this branch.
  • Removed the drafts/github-readmes/* files — out of scope for this PR.
  • Reverted an unrelated src/utils/highlightCode.ts refactor that had been mixed into this branch; it will be proposed separately.
  • Rewrote src/sections/Team.astro: the two-column layout (contributor grid beside a "want to contribute" sidebar) is replaced with a single centered column — heading, then a responsive contributor grid (up to 4 columns), then a "want to contribute" block. Contributor cards now link the avatar and name to contributor.github_url when set, falling back to plain (non-linked) rendering otherwise; organisation is only rendered when present. The single "View all contributors" button is replaced with two buttons, one linking to ${GITHUB_CODECARBON_URL}/graphs/contributors and one to ${GITHUB_ECOLOGITS_URL}/graphs/contributors (both imported from ../consts).
  • Copy changes: the Welcome hero's h1 now reads "Measure the environmental footprint of compute & GenAI" (was "computing & AI").

Handle confirmation

Each handle was confirmed against a GitHub user profile whose name field matches the contributor's name (or, where the profile is already referenced via img_path's avatar id, the id-derived login's name):

  • Benoît Courty → benoit-cty
  • Amine Saboni → SaboniAmine
  • Sasha Luccioni → sashavor
  • Iñigo Imaz → inimaz
  • Luis Blanche → LuisBlanche
  • Patrick → prmths128 (profile name: "Patrick LLORET")
  • Claire Saignol → Claire6675 (profile name: "Claire SAIGNOL")
  • Valentin Defour → left out; the avatar id resolves to vdwow (profile name: "Valentin D."), which only weakly matches — not confirmed.

Related Issue

N/A

Motivation and Context

@inimaz asked to drop the draft READMEs and to fill in GitHub URLs for the remaining contributors.

How Has This Been Tested?

  • pnpm install
  • pnpm run build (astro build, succeeds)
  • npx prettier --check on the changed files

Screenshots (if appropriate):

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Surface EcoLogits maintainers from the docs site alongside CodeCarbon volunteers, including David Berenstein, with project badges and GitHub links.
@davidberenstein1957
davidberenstein1957 requested a review from a team as a code owner August 3, 2026 08:51
@davidberenstein1957

Copy link
Copy Markdown
Contributor Author

Hey @inigoimaz — would love your review on this when you have a moment 🙂

Removed organisation details for contributors and enhanced the layout for better responsiveness and visual appeal. Adjusted text alignment and spacing for improved readability.
Updated the highlighter initialization to use global variables for better performance and to avoid redundant instances. This change allows for asynchronous loading of the highlighter while ensuring a single instance is reused across calls.
…d Team.astro

Eliminated the 'products' field from the ContributorSchema and updated the Team.astro section to reflect this change, enhancing the clarity and focus on contributor details.

@inimaz inimaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @davidberenstein1957 for this, left some comments

Comment thread drafts/github-readmes/README.md Outdated
Comment thread src/content/contributors.ts

@inimaz inimaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@inimaz
inimaz merged commit d32e549 into main Sep 28, 2026
1 check passed
@inimaz
inimaz deleted the feature/ecologits-contributors branch September 28, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants