Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Jul 9, 2025

This PR addresses the contributor listing sorting requirements by analyzing the current state and adding validation.

Analysis Results

After thoroughly examining both README.md and CONTRIBUTING.md, I found that:

  • All contributor listings are already sorted alphabetically by first name
  • No company affiliations exist in emeritus sections that need removal
  • CONTRIBUTING.md contains no contributor listings to sort

The current state of README.md already meets all requirements:

  • Maintainers section: Correctly sorted (Alex → Andrzej → Antoine → Bogdan → Dmitrii → Evan → Pablo → Sean → Tyler → Yang)
  • Approvers section: Correctly sorted (Christos → Curtis → David → John → Matt → Moritz → Sam → Ziqi)
  • Emeritus Maintainers section: Correctly sorted (Daniel → Juraci → Tigran) with no company affiliations
  • Emeritus Approvers section: Correctly sorted (Anthony → Bryan → Przemek → Ruslan) with no company affiliations

Changes Made

Since the contributor listings were already properly formatted, I've added a comprehensive Go test (tests/contributor_test.go) to ensure this format is maintained in the future. The test validates:

  1. All contributor sections are sorted alphabetically by first name (case-insensitive)
  2. Emeritus sections contain no company affiliations
  3. Proper handling of special name cases (e.g., "John L. Peterson (Jack)")

The test runs successfully and integrates with the existing test infrastructure.

Fixes #29.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: trask <218610+trask@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Sort contributor listings Add test to validate contributor listing format and confirm current state Jul 9, 2025
@Copilot Copilot AI requested a review from trask July 9, 2025 15:20
Copilot finished work on behalf of trask July 9, 2025 15:20
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.

Sort contributor listings

2 participants