Skip to content

Conversation

earth2marsh
Copy link
Member

Following the provisional period, welcome to the new maintainers! Thank you for all the work you've put in until now and for the progress yet to come. 🙏

Following the provisional period, welcome to the new maintainers! Thank you for all the work you've put in until now and for the progress yet to come. 🙏
@earth2marsh earth2marsh requested a review from a team as a code owner September 25, 2024 00:30
Copy link
Contributor

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

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

Do we want to sort by first name?

@ralfhandl ralfhandl requested a review from a team September 25, 2024 07:07
@earth2marsh
Copy link
Member Author

Do we want to sort by first name?

Hmmm, fair question! I think there are two general implicit principles that could be made explicit.

  1. The emeritus section has had names added (prepended) at the top of the list as members move out of active status (a sort of trickling down from above).
  2. The active section has had new members appended. When a cohort contains more than one member, it is sorted by first name.

Of course that's just the organic way it has evolved! Given that the active list is going to be a fairly small number, the value of sorting it alphabetically by first name might not be that critical? But if it makes more sense to people viewing it, I'm not opposed!

Copy link
Contributor

@mikekistler mikekistler left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@ralfhandl ralfhandl merged commit 0e3eec8 into main Sep 26, 2024
4 checks passed
@ralfhandl ralfhandl deleted the tsc-2024 branch September 26, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants