Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix grammar, remove duplicates and add missing punctuation #76

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

savvar9991
Copy link

  1. Fixed grammar in [!NOTE] (line 51):
  • zod` is marked as optional peer dependency, so if you all you need is the JSON Schema definitions
  • zod` is marked as optional peer dependency, so if all you need is the JSON Schema definitions

Reason: Removed redundant "you" for proper grammar.

2.Added punctuation (line 133):

  • Post metadata schema is a union of all content schemas (e.g. ArticleMetadata, AudioMetadata, etc.
  • Post metadata schema is a union of all content schemas (e.g. ArticleMetadata, AudioMetadata, etc.).

Reason: Added a period at the end of the sentence for proper punctuation.

3.Improved grammar (line 201):

  • Every time you have a discriminated union, you can use the discriminant to narrow the type. See few examples below.
  • Every time you have a discriminated union, you can use the discriminant to narrow the type. See a few examples below.

Reason: Added the article "a" before "few examples" for proper grammar.

4.Removed duplication (line 479):

  • You can you can use ESModule aliasing to avoid conflicts
  • You can use ESModule aliasing to avoid conflicts

Reason: Removed duplicated "you can" to fix redundancy.

Copy link

height bot commented Dec 15, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

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