mieliespoor commented on PR #8334: URL: https://github.com/apache/incubator-devlake/pull/8334#issuecomment-2746389600
Added one more comment around readability and consistency. Looks good on a highlevel. Two things I would say that is missing on this PR: 1. Some screenshots on how the UI would look after the changes? 2. Missing unit tests. No unit tests were added or changed as part of this. Other than that, I would allow the code owners to also review this. -- 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]
