adamsaghy commented on PR #4322: URL: https://github.com/apache/fineract/pull/4322#issuecomment-2678243578
> Thanks for the feedback @adamsaghy ! I understand the need to increase the scope of PRs to maintain a cleaner commit history. > > For this PR, I initially focused on refactoring `AcmeNoopJobConfiguration` to ensure the changes were isolated and easy to review. However, I agree that a broader refactor across multiple modules would be more efficient. > > Would you prefer I extend this PR to cover additional modules, or should we merge this first and then follow up with a larger refactor in a separate PR? Let me know how you'd like to proceed. Its okay to do it in the future ;) -- 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]
