Skip to content

chore: fix broken TODO comment with accidental URL#5512

Open
Zentex1337 wants to merge 1 commit intoreown-com:mainfrom
Zentex1337:fix/broken-todo-comment
Open

chore: fix broken TODO comment with accidental URL#5512
Zentex1337 wants to merge 1 commit intoreown-com:mainfrom
Zentex1337:fix/broken-todo-comment

Conversation

@Zentex1337
Copy link

Summary

  • Clean up a TODO comment in ThemeConstantsUtil.ts that had a localhost URL accidentally pasted into it
  • The original comment @TODO: Provide proper values to something sm, md, lg, http://localhost:6006/?path=/docs/theme-spacing--docsxl, etc. is now cleaned to @TODO: Provide proper values using named sizes (sm, md, lg, xl, etc.)

Test plan

  • Comment-only change, no functional impact
  • Improves code readability

The TODO comment had a localhost URL accidentally pasted into it,
making the comment hard to read. Cleaned up to be more concise.
@vercel
Copy link

vercel bot commented Feb 3, 2026

@Zentex1337 is attempting to deploy a commit to the Reown Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link

changeset-bot bot commented Feb 3, 2026

⚠️ No Changeset found

Latest commit: 382f5f2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

Thank you for your contribution! We ask that you please read and sign our CTA Document before we can accept your contribution. You can sign the CTA simply by posting a Pull Request Comment with the following text:


I have read the CTA Document and I hereby sign the CTA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant