Skip to content

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Jan 14, 2026

Applies the flex codemod to files owned by @getsentry/explore.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 14, 2026
@@ -1,18 +1,16 @@
import styled from '@emotion/styled';

import type {FlexProps} from 'sentry/components/core/layout';
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cursoragent remove the duplicate flex import

Copy link
Contributor

@cursor cursor bot Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done! I've removed the duplicate FlexProps import from static/app/views/discover/table/quickContext/styles.tsx. The file was importing FlexProps twice - once on line 3 as a standalone type import, and again on line 4 alongside Flex and Stack.

I kept the single import statement that includes all three (Flex, Stack, and type FlexProps) and removed the redundant one. The fix has been committed and pushed to the codemod-migration/getsentry-explore branch, which should resolve the TypeScript error.


Open in Cursor Open in Web

@JonasBa JonasBa marked this pull request as ready for review January 14, 2026 20:31
@JonasBa JonasBa requested a review from a team as a code owner January 14, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants