bogdanpolak / bogdanpolak/command-delphi

Remove DoGuard from TCommand

Open
#52 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Pascal
Stars
29
Forks
6
PR merge metrics
No merged PRs in 30d

Description

1) Remove: DoGuard from TCommand
1) Add documentation: to check all mandatory injections with `Assert` calls at the very begging of the `DoExecute` method

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate TCommand and its DoExecute method, then inspect how DoGuard and mandatory injections are used. Remove DoGuard, document the required Assert checks at the beginning of DoExecute, and verify that the resulting documentation matches the mandatory injections.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.