Skip to content

Conversation

@dchansen06
Copy link
Contributor

Running !users outputs the following

Discord Members: 875
Members Role: 629
Alumni Members: 22Firmware: 80
Electronics: 137
Suspension: 47
Data: 58
Aero: 61
Systems: 11
Chassis: 48
Drivetrain: 55
Business: 28

Leads: 18
Officers: 7
Special Advisors: 10

This PR adds a newline after the Alumni Members and before the subteam enumeration to fix that line's formatting.

@dchansen06 dchansen06 self-assigned this Oct 23, 2025
@dchansen06 dchansen06 added bug Something isn't working enhancement New feature or request labels Oct 23, 2025
@netlify
Copy link

netlify bot commented Oct 23, 2025

Deploy Preview for gr-sentinel canceled.

Name Link
🔨 Latest commit fff805a
🔍 Latest deploy log https://app.netlify.com/projects/gr-sentinel/deploys/68f97e4bdcf0ee0009185e67

@dchansen06 dchansen06 requested a review from Copilot October 23, 2025 01:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fixes a formatting issue in the !users command output where Alumni Members and the first subteam (Firmware) were concatenated on the same line instead of appearing on separate lines.

Key Change:

  • Added a newline character after the Alumni Members count in the output format string

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dchansen06 dchansen06 requested a review from BK1031 October 23, 2025 01:03
Copy link
Contributor

@BK1031 BK1031 left a comment

Choose a reason for hiding this comment

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

Lgtm

@dchansen06 dchansen06 merged commit be8020e into main Oct 23, 2025
7 checks passed
@dchansen06 dchansen06 deleted the AddNewlineAfterAlumniMembersInUsers branch October 23, 2025 01:16
@dchansen06
Copy link
Contributor Author

@austinchan3678 Has this made it into the current release?

Discord

Discord Members: 866
Members Role: 618
Alumni Members: 23Data: 55
Drivetrain: 54
Aero: 58
Electronics: 134
Chassis: 47
Suspension: 47
Business: 28
Firmware: 72
Systems: 10

Leads: 17
Officers: 7
Special Advisors: 9

@BK1031
Copy link
Contributor

BK1031 commented Dec 5, 2025

lol i would hella hope so

@BK1031
Copy link
Contributor

BK1031 commented Dec 5, 2025

but that also reminds me we gotta hella migrate off of docker hub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants