Skip to content

Conversation

@WA11AX
Copy link
Owner

@WA11AX WA11AX commented Aug 13, 2025

Summary

  • remove automatic call to initializeData from storage
  • add dedicated seeding script and npm command
  • document how to seed demo tournaments

Testing

  • npm test
  • npm run lint:check
  • npm run type-check

https://chatgpt.com/codex/tasks/task_e_689cd877c9dc83279fb651eb3560cbb4

Copilot AI review requested due to automatic review settings August 13, 2025 18:33
Copy link
Contributor

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

This PR removes automatic initialization of demo tournament data and introduces a dedicated seeding script for better control over when sample data is populated. The changes provide a cleaner separation between application startup and data seeding.

  • Removes automatic initializeData call from storage module
  • Adds dedicated seeding script with proper error handling and exit codes
  • Documents the new seeding process in README files

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
server/storage.ts Removes automatic initialization call that was running on startup
server/seed.ts New dedicated seeding script with error handling and process management
server/README.md Documents the new npm seed command
package.json Adds npm script for running the seeding process
README.md Updates setup instructions to include optional seeding step

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@WA11AX WA11AX merged commit 6039fd5 into main Aug 13, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants