Skip to content

Refactor validation and path helpers#91

Open
darshan-Jahagirdar wants to merge 1 commit into
ApexOpsStudio:mainfrom
darshan-Jahagirdar:codex/refactor-validation-utils
Open

Refactor validation and path helpers#91
darshan-Jahagirdar wants to merge 1 commit into
ApexOpsStudio:mainfrom
darshan-Jahagirdar:codex/refactor-validation-utils

Conversation

@darshan-Jahagirdar
Copy link
Copy Markdown

@darshan-Jahagirdar darshan-Jahagirdar commented May 13, 2026

Fixes #3.

Changes

  • Added utils/validation.py for shared validation helpers.
  • Added utils/paths.py for the shared task storage path helper.
  • Updated add/list/done commands to import the shared helpers.
  • Updated tests to import validators from the new shared module.

Verification

  • python -m pytest test_task.py -> 2 passed
  • rg "def validate_|def get_tasks_file|Validation logic scattered" commands utils test_task.py shows helper definitions only under utils/

/claim #3

@darshan-Jahagirdar
Copy link
Copy Markdown
Author

Validation update for this PR:

ext py -3.11 -B -m pytest test_task.py git diff --check

Result: 2 tests passed and the diff whitespace check passed. I installed pytest into a temporary test-only directory for this run and removed it afterward.

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.

Refactor: Extract validation to utils module

1 participant