Copilot commented on code in PR #8998: URL: https://github.com/apache/texera/pull/8998#discussion_r4284986750
########## LICENSE: ########## @@ -243,8 +243,10 @@ This product bundles the ScalaPB options definition: Source: https://github.com/scalapb/ScalaPB License: Apache License 2.0 (this LICENSE file) -This product includes an icon from Google's Material Symbols: +This product includes icons from Google's Material Symbols: - frontend/src/app/common/icon/model-icon.ts ("hub") + - the "psychology" and "smart_toy" icons drawn into + frontend/src/assets/notebook_migration_tool/tool_popup_diagram.png Review Comment: The Material Symbols entry mixes a source code file path (`model-icon.ts`) with asset references, which makes it unclear what exact Material Symbols assets are being redistributed. Consider listing the actual icon identifiers consistently (e.g., `"hub"`, `"psychology"`, `"smart_toy"`) and, if helpful, point to where they are used (code path) separately (e.g., “used by …”), so the LICENSE clearly enumerates the third-party content rather than implementation details. -- 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]
