Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@LekoArts
Copy link
Contributor

@LekoArts LekoArts commented Apr 15, 2025

Description

In this PR I'm normalizing our usage of the @default, @deprecated tags. I'm also modifying the Typedoc output to sometimes adjust the @returns tag output.

I've checked our codebase for instances of "Defaults to ..." and replaced it with @default <val>. This way it's captured correctly and can be added in our Typedoc output in a coherent way.

Same thing with @deprecated. Sometimes a full stop was missing at the end, sometimes the first word wasn't capitalized. I've changed the wording to have the sentence structure of "Use X instead.".

I have also mostly removed sentences inside @deprecated that said "This is deprecated" since it's unnecessary so say that inside the tag. I've also sometimes removed sentences like "Will be removed in next major version" because I don't think it adds much value, it's deprecated nevertheless.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot
Copy link

changeset-bot bot commented Apr 15, 2025

🦋 Changeset detected

Latest commit: e3535e1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 22 packages
Name Type
@clerk/react-router Patch
@clerk/clerk-js Patch
@clerk/backend Patch
@clerk/shared Patch
@clerk/astro Patch
@clerk/clerk-react Patch
@clerk/remix Patch
@clerk/types Patch
@clerk/clerk-expo Patch
@clerk/vue Patch
@clerk/chrome-extension Patch
@clerk/agent-toolkit Patch
@clerk/express Patch
@clerk/fastify Patch
@clerk/nextjs Patch
@clerk/nuxt Patch
@clerk/tanstack-react-start Patch
@clerk/testing Patch
@clerk/elements Patch
@clerk/expo-passkeys Patch
@clerk/localizations Patch
@clerk/themes Patch

Not sure what this means? Click here to learn what changesets are.

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

@vercel
Copy link

vercel bot commented Apr 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
clerk-js-sandbox ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 9:00am

@LekoArts LekoArts changed the title chore(*): Improve @default and @returns JSDoc usage chore(*): Improve @default, @deprecated, & @returns JSDoc Apr 15, 2025
@LekoArts LekoArts changed the title chore(*): Improve @default, @deprecated, & @returns JSDoc chore(*): Improve @default, @deprecated JSDoc Apr 15, 2025
@LekoArts LekoArts marked this pull request as ready for review April 15, 2025 09:15
@LekoArts LekoArts merged commit 7389ba3 into main Apr 15, 2025
47 checks passed
@LekoArts LekoArts deleted the lekoarts/eco-598-second-review-pass branch April 15, 2025 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants