chore: cleanup migration + duplicate#1433
Conversation
jakobmoellerdev
left a comment
There was a problem hiding this comment.
Whats the reason for deprecating the OCM internal utils in favor of mandelsoft? to me this is undesired during a cleanup and duplication removal that is cross-package. We should only upgrade usages that already exist.
|
The functions are duplicates, because during the migration not all occurrences have been adapted. The alternative is to migrate back this completely to the ocm library. But we should have only one source of truth for those functions. |
In theory, I agree. But using libraries from private contributors or without community support is risky. What happens if the contributor no longer has access to the repositories or makes them private? That can lead to much bigger problems. |
Let's do that because of the above, what Frederic mentioned. :) |
|
This pull request was marked as stale because it has not had recent activity. |
What this PR does / why we need it
Which issue(s) this PR fixes