Refactor #2073
closed
The behavior of the check_publish function does not match its name or description
0%
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.
Updated by bmbouter about 5 years ago
- Status changed from NEW to CLOSED - WONTFIX
Updated by bmbouter about 5 years ago
Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.
.