Skip to content

Miscellanous changes#78

Open
tmcnulty387 wants to merge 8 commits intoComputerScienceHouse:devfrom
tmcnulty387:dev
Open

Miscellanous changes#78
tmcnulty387 wants to merge 8 commits intoComputerScienceHouse:devfrom
tmcnulty387:dev

Conversation

@tmcnulty387
Copy link
Member

What

Small miscellanous changes that don't affect core functionality

  • Updated gitignore to ignore .vscode
  • Replaced context.TODO() with context.Background()
  • Fixed incorrect method names in logs
  • Small changes suggested by go compiler
  • Added godotenv to automatically load .env if it exists
  • Explicitly connect to database from main instead of from database.go

Why

Because why not

Test Plan

go test -v ./... passes and none of the changes affects core functionality

Env Vars

did you add, remove, or rename any environment variables
No

Checklist

  • Tested all changes locally

@tmcnulty387 tmcnulty387 changed the title Dev Miscellanous changes Feb 16, 2026
@tmcnulty387
Copy link
Member Author

1 of the changes suggested by compiler was wrong, replacing interface{} with any did break stuff. Reverted that change.

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.

1 participant