Skip to content

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Jan 14, 2026

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

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 14, 2026
display: flex;
gap: ${space(1)};
`;
export function ButtonWrapper(props: FlexProps<'span'>) {
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 modify this to omit as prop from flex signature

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.

Perfect! I've successfully modified the ButtonWrapper component to omit the as prop from the Flex signature.

What I did:

  • Changed the props type from FlexProps<'span'> to Omit<FlexProps<'span'>, 'as'>
  • This resolves the TypeScript error by excluding the as prop from the component's signature, since it's already hardcoded to "span" in the implementation

The change has been committed and pushed to the codemod-migration/getsentry-ecosystem branch. This should fix the TypeScript CI failure.


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