Skip to content

Conversation

@Kludex
Copy link
Member

@Kludex Kludex commented Jan 30, 2026

No description provided.

@claude
Copy link

claude bot commented Jan 30, 2026

Code review

Issue: Placeholder email addresses

Two of the new maintainer entries are using placeholder email addresses instead of real email addresses:

  • Line 12: { name = "Max Isbey", email = "add-yourself@gmail.com" }
  • Line 13: { name = "Felix Weinberger", email = "add-yourself@gmail.com" }

The email add-yourself@gmail.com is clearly a placeholder value that should be replaced with the actual email addresses for Max Isbey and Felix Weinberger before merging.

Why this matters:

  • This metadata will be published to PyPI
  • Both maintainers have the identical placeholder email, which is not legitimate
  • The existing maintainers all have proper, distinct email addresses
  • This creates confusion and makes it impossible to contact these maintainers

Suggested fix:
Max Isbey and Felix Weinberger should provide their actual email addresses to replace the placeholder values.

See the code here:

{ name = "Marcelo Trylesinski", email = "marcelotryle@gmail.com" },
{ name = "Max Isbey", email = "add-yourself@gmail.com" },
{ name = "Felix Weinberger", email = "add-yourself@gmail.com" }

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.

3 participants