korbit-ai[bot] commented on code in PR #32226:
URL: https://github.com/apache/superset/pull/32226#discussion_r1953206576
##########
superset-frontend/src/dashboard/components/menu/MarkdownModeDropdown.tsx:
##########
@@ -45,7 +45,9 @@ export default class MarkdownModeDropdown extends
PureComponent<MarkdownModeDrop
const { id, value, onChange } = this.props;
return (
+ // Add data-testid to the PopoverDropdown component
Review Comment:
### Redundant Inline Comment <sub></sub>
<details>
<summary>Tell me more</summary>
###### What is the issue?
The inline comment explaining the addition of a data-testid is unnecessary
and redundant as the code change is self-documenting.
###### Why this matters
Redundant comments increase noise and make the code harder to read without
adding any valuable information. For self-documenting code changes like adding
a data-testid, no comment is needed.
###### Suggested change ∙ *Feature Preview*
Remove the comment line:
```typescript
<PopoverDropdown
data-testid="markdown-mode-dropdown"
id={id}
options={dropdownOptions}
value={value}
onChange={onChange}
/>
```
</details>
<sub>
[](https://app.korbit.ai/feedback/aa91ff46-6083-4491-9416-b83dd1994b51/3ed6f86e-c27c-42ea-b894-4dd79166d6b2?suggestedFixEnabled=true)
💬 Chat with Korbit by mentioning @korbit-ai.
</sub>
<!--- korbi internal id:ae1e39ee-be97-4fe0-b706-ea45e86e57c0 -->
--
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: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
