Documentation Fix
File: internal/runtime/condition.go, line 23
Current comment:
// Error message for the case not all condition was not met
Problem: This comment contains a double-negative: "not all condition was not met". The two negatives cancel each other out, making the comment semantically incorrect and confusing to read.
Fix:
// Error message for when not all conditions are met
This makes the comment clear, grammatically correct (fixes "condition" → "conditions" plural), and removes the double-negative.
Additional: The comment also uses "condition" (singular) when the context refers to multiple conditions, consistent with the function EvalConditions which handles a list.
Documentation Fix
File:
internal/runtime/condition.go, line 23Current comment:
// Error message for the case not all condition was not metProblem: This comment contains a double-negative: "not all condition was not met". The two negatives cancel each other out, making the comment semantically incorrect and confusing to read.
Fix:
// Error message for when not all conditions are metThis makes the comment clear, grammatically correct (fixes "condition" → "conditions" plural), and removes the double-negative.
Additional: The comment also uses "condition" (singular) when the context refers to multiple conditions, consistent with the function
EvalConditionswhich handles a list.