Copilot commented on code in PR #38471:
URL: https://github.com/apache/superset/pull/38471#discussion_r2897267009
##########
superset-frontend/plugins/legacy-plugin-chart-world-map/src/WorldMap.ts:
##########
@@ -301,32 +301,6 @@ function WorldMap(element: HTMLElement, props:
WorldMapProps): void {
.selectAll('.datamaps-subunit')
.on('contextmenu', handleContextMenu)
.on('click', handleClick)
Review Comment:
The D3 event handler chain in `done` no longer terminates with a semicolon
after removing the hover handlers. With Superset's Prettier-enforced
formatting, this will likely cause `prettier-check`/lint failures; add a
trailing semicolon after the final `.on('click', handleClick)` call for
consistency with the rest of the file.
```suggestion
.on('click', handleClick);
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
