e2corporation commented on PR #2552: URL: https://github.com/apache/incubator-devlake/pull/2552#issuecomment-1192777269
@likyh Thanks for the PR, as we discussed there are some very valuable changes proposed here, I especially like the addition of the Transformation Manager Hook, that will be useful in the future for better encapsulation and reduce the coupling concerns you raised earlier. Since my last service pack 1.0 PR for UI has resolved a few of the key issues that this ticket originally aimed to fix, we'll hold on merging for now. We will collaborate on incorporating some of these ideas in a future ticket when the transformation logic is extracted. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
