doorstop-dev / doorstop

Requirements management using version control.
https://doorstop.readthedocs.io
Other
466 stars 128 forks source link

Link to inactive item causes ERROR, not WARNING, when checking Child's link to Parent #575

Closed robertberrington closed 2 years ago

robertberrington commented 2 years ago

Changing a parent item "active=false" causes an ERROR when running doorstop to validate when checking the Child's link to the Parent. Documentation says this is a WARNING.

Reference: https://doorstop.readthedocs.io/en/latest/cli/validation/

An item is linked to an inactive item.

I have three levels ... Grandparent, Parent, Child. When checking the Grandparent <- Parent to verify all Grandparent items have child (Parent) links, I get a WARNING. When checking Child -> Parent, I get an ERROR.

If the behavior is correct, documentation should say:

WARNING:  An item's child link is an inactive item.
ERROR:  An item's parent link is an inactive item.
jacebrowning commented 2 years ago

I think the behavior is correct and the documentation should reflect this. The rationale is that "upward" links are the responsibility of that item (via ERROR) but it is useful to know that a "downward" link (child) has been marked inactive (via WARNING).