Actions
Refactor #2073
closedThe behavior of the check_publish function does not match its name or description
Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:
0%
Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:
Description
check_publish() is described as checking whether a publish operation should be skipped or not, which it does, but it also then performs a publish itself and returns the result.
All of the publishing logic should be moved to the publish() function in the same module, which should utilize check_publish only to determine if the publish should take place or if it can be skipped.
Actions