Skip to content

Moved routes into their own functions#100

Open
pikachu0542 wants to merge 1 commit intodevfrom
routes-to-functions
Open

Moved routes into their own functions#100
pikachu0542 wants to merge 1 commit intodevfrom
routes-to-functions

Conversation

@pikachu0542
Copy link
Copy Markdown
Member

What

Moved each route into its own function

Why

Not having a giant main function with all the logic is good

Test Plan

Ensure that all functionality works as expected

Env Vars

No

Checklist

  • Tested all changes locally

@pikachu0542 pikachu0542 self-assigned this Mar 29, 2026
@pikachu0542 pikachu0542 added enhancement New feature or request go Pull requests that update go code labels Mar 29, 2026
Copy link
Copy Markdown
Contributor

@tallen42 tallen42 left a comment

Choose a reason for hiding this comment

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

this is awesome

Copy link
Copy Markdown
Contributor

@costowell costowell left a comment

Choose a reason for hiding this comment

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

I really like this already. It might be good to move these calls to a separate file/module to de-clutter main.go ( 🥭 )

@csh-sonarqube-community
Copy link
Copy Markdown

Failed Quality Gate failed

  • 0.00% Coverage on New Code (is less than 80.00%)

Project ID: ComputerScienceHouse_vote_c5ba863d-30d7-4fa9-97dd-4f2c58f8f5fa

View in SonarQube

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

Labels

enhancement New feature or request go Pull requests that update go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants