Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Oct 2, 2025

Summary

This PR implements a new Config class to centralize configuration options for the FishE game, addressing issue #[issue_number].

Changes

New Files

  • src/config/config.py: Contains the Config class with configuration attributes for:

    • Save file paths (data directory, player.json, stats.json, timeService.json)
    • Initial player values (starting money, energy, fish count, bank balance, fish multiplier, bait price)
  • tests/config/test_config.py: Unit tests validating all Config class attributes

  • src/config/__init__.py: Package initialization file

Modified Files

  • .gitignore: Added __pycache__/ to exclude Python cache directories

Configuration Options

The Config class provides 10 configuration attributes:

File Paths:

  • dataDirectory: Directory for save files
  • playerSaveFile: Player state save file path
  • statsSaveFile: Statistics save file path
  • timeServiceSaveFile: Time service save file path

Initial Player Values:

  • initialMoney: Starting money ($20)
  • initialEnergy: Starting energy (100)
  • initialFishCount: Starting fish count (0)
  • initialMoneyInBank: Starting bank balance ($0.01)
  • initialFishMultiplier: Starting fish multiplier (1)
  • initialPriceForBait: Starting bait price ($50)

Usage Example

from src.config.config import Config

config = Config()
print(f"Player starts with ${config.initialMoney}")
print(f"Save files location: {config.dataDirectory}")

Testing

All 63 tests pass, including new tests for the Config class. The implementation follows existing code patterns and does not modify any existing functionality.

Original prompt

This section details on the original issue you should resolve

<issue_title>Create a Config class.</issue_title>
<issue_description>A class to hold config options should be created.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #14

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: dmccoystephenson <21204351+dmccoystephenson@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Create a Config class. Create Config class to hold configuration options Oct 2, 2025
@Copilot Copilot AI requested a review from dmccoystephenson October 2, 2025 06:22
Copilot finished work on behalf of dmccoystephenson October 2, 2025 06:22
@dmccoystephenson dmccoystephenson marked this pull request as ready for review October 2, 2025 06:25
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.

Create a Config class.
2 participants