Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for five digit team numbers #2655

Merged
merged 4 commits into from
May 17, 2024

Conversation

sciencewhiz
Copy link
Collaborator

No description provided.

@jasondaming
Copy link
Member

I suggested to modify the traditional TE.AM to XTE.AM, what do you think? @sciencewhiz
https://www.chiefdelphi.com/t/team-numbers-have-officially-hit-5-digits-what-now/466213/24

@PeterJohnson
Copy link
Member

I liked the suggestion of 10.STE.AM.x later in the thread.

@jasondaming
Copy link
Member

I was worried that would change the "obviousness" that it was the team number. I do think that would prevent confusion around 10.XTE.AM.X which less ideally uses X twice (we could also change that variable, we already use Y and Z in other places). Either way I think both of these are much better than just TE.AM

@sciencewhiz
Copy link
Collaborator Author

I don't like STE.AM, it uses something unrelated in a confusing way.

I think TE.AM with explanation is sufficient, but if I were to choose something else it'd prefer TEA.M#

@PeterJohnson
Copy link
Member

I think just sticking with TE.AM works too with a simple explanation that "AM" is always the last two digits of the team number, so "TE" is 3 digits for 5-digit teams.

Co-authored-by: Jason Daming <jason.daming@intralox.com>
@jasondaming jasondaming merged commit 3e37c9e into wpilibsuite:main May 17, 2024
10 checks passed
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.

None yet

3 participants