-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Milestone
Description
Description
Currently instructions are split across two files:
SimpleInstruction.py- Instructions without argumentsInstructionWithArguments.py- Instructions with arguments
This separation is arbitrary and makes the code harder to navigate. Consolidate into a single instructions.py file.
Files to Merge
instruction/SimpleInstruction.pyinstruction/InstructionWithArguments.py
Target
instruction/instructions.py(new file)
Acceptance Criteria
- Create new
instruction/instructions.pywith all instruction classes - Use consistent base class pattern for all instructions
- Delete
SimpleInstruction.pyandInstructionWithArguments.py - Update
instruction/__init__.pyexports - All imports throughout codebase updated
- All existing tests pass
Dependencies
- Depends on Create unified Instruction protocol and fix inheritance #58 (unified Instruction protocol)
Metadata
Metadata
Assignees
Labels
No labels