-
Notifications
You must be signed in to change notification settings - Fork 872
chore(docs): renovate guides #2722
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
base: canary
Are you sure you want to change the base?
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 issues found across 9 files
Prompt for AI agents (all 4 issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="apps/docs/getting-started/monorepo-setup/yarn.mdx">
<violation number="1" location="apps/docs/getting-started/monorepo-setup/yarn.mdx:64">
P2: File name inconsistency: The code block now references `email.tsx` but the surrounding documentation (step 5 prose and step 7) still references `MyEmail.tsx`. Consider updating step 5 to say "create a file inside called `email.tsx`" and step 7 to reference `emails/email.tsx`.</violation>
</file>
<file name="apps/docs/getting-started/monorepo-setup/pnpm.mdx">
<violation number="1" location="apps/docs/getting-started/monorepo-setup/pnpm.mdx:48">
P2: Documentation inconsistency: The code block path changed to `email.tsx`, but Step 4's text still instructs creating `MyEmail.tsx` and Step 6 references editing `emails/MyEmail.tsx`. Consider updating the prose to match the new filename or keeping the original filename.</violation>
</file>
<file name="apps/docs/getting-started/monorepo-setup/bun.mdx">
<violation number="1" location="apps/docs/getting-started/monorepo-setup/bun.mdx:49">
P2: Documentation inconsistency: The code block filename was changed to `email.tsx` but the instruction text above still says to create `MyEmail.tsx`. Users following this guide will be confused about which filename to use. Consider updating the instruction text to match the new filename.</violation>
</file>
<file name="apps/docs/getting-started/monorepo-setup/npm.mdx">
<violation number="1" location="apps/docs/getting-started/monorepo-setup/npm.mdx:49">
P2: Documentation inconsistency: The code block path was changed to `email.tsx` but the prose in Step 4 still instructs users to create `MyEmail.tsx`, and Step 6 still references `emails/MyEmail.tsx`. Either the prose descriptions should be updated to match the new filename, or the code block path should remain as `MyEmail.tsx`.</violation>
</file>
Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR
|
|
||
| ```jsx packages/transactional/emails/MyEmail.tsx | ||
| import { Button, Html } from "@react-email/components"; | ||
| ```jsx packages/transactional/emails/email.tsx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: File name inconsistency: The code block now references email.tsx but the surrounding documentation (step 5 prose and step 7) still references MyEmail.tsx. Consider updating step 5 to say "create a file inside called email.tsx" and step 7 to reference emails/email.tsx.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/docs/getting-started/monorepo-setup/yarn.mdx, line 64:
<comment>File name inconsistency: The code block now references `email.tsx` but the surrounding documentation (step 5 prose and step 7) still references `MyEmail.tsx`. Consider updating step 5 to say "create a file inside called `email.tsx`" and step 7 to reference `emails/email.tsx`.</comment>
<file context>
@@ -61,24 +61,25 @@ Include the following script in your `package.json` file.
-```jsx packages/transactional/emails/MyEmail.tsx
-import { Button, Html } from "@react-email/components";
+```jsx packages/transactional/emails/email.tsx
+import { Button, Html, Head, Body } from "@react-email/components";
import * as React from "react";
</file context>
|
|
||
| ```jsx packages/transactional/emails/MyEmail.tsx | ||
| import { Button, Html } from "@react-email/components"; | ||
| ```jsx packages/transactional/emails/email.tsx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Documentation inconsistency: The code block path changed to email.tsx, but Step 4's text still instructs creating MyEmail.tsx and Step 6 references editing emails/MyEmail.tsx. Consider updating the prose to match the new filename or keeping the original filename.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/docs/getting-started/monorepo-setup/pnpm.mdx, line 48:
<comment>Documentation inconsistency: The code block path changed to `email.tsx`, but Step 4's text still instructs creating `MyEmail.tsx` and Step 6 references editing `emails/MyEmail.tsx`. Consider updating the prose to match the new filename or keeping the original filename.</comment>
<file context>
@@ -45,24 +45,25 @@ Include the following script in your `package.json` file.
-```jsx packages/transactional/emails/MyEmail.tsx
-import { Button, Html } from "@react-email/components";
+```jsx packages/transactional/emails/email.tsx
+import { Button, Html, Head, Body } from "@react-email/components";
import * as React from "react";
</file context>
| ```jsx packages/transactional/emails/email.tsx | |
| ```jsx packages/transactional/emails/MyEmail.tsx |
|
|
||
| ```jsx packages/transactional/emails/MyEmail.tsx | ||
| import { Button, Html } from "@react-email/components"; | ||
| ```jsx packages/transactional/emails/email.tsx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Documentation inconsistency: The code block filename was changed to email.tsx but the instruction text above still says to create MyEmail.tsx. Users following this guide will be confused about which filename to use. Consider updating the instruction text to match the new filename.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/docs/getting-started/monorepo-setup/bun.mdx, line 49:
<comment>Documentation inconsistency: The code block filename was changed to `email.tsx` but the instruction text above still says to create `MyEmail.tsx`. Users following this guide will be confused about which filename to use. Consider updating the instruction text to match the new filename.</comment>
<file context>
@@ -46,24 +46,25 @@ Include the following script in your `package.json` file.
-```jsx packages/transactional/emails/MyEmail.tsx
-import { Button, Html } from "@react-email/components";
+```jsx packages/transactional/emails/email.tsx
+import { Button, Html, Head, Body } from "@react-email/components";
import * as React from "react";
</file context>
|
|
||
| ```jsx packages/transactional/emails/MyEmail.tsx | ||
| import { Button, Html } from "@react-email/components"; | ||
| ```jsx packages/transactional/emails/email.tsx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Documentation inconsistency: The code block path was changed to email.tsx but the prose in Step 4 still instructs users to create MyEmail.tsx, and Step 6 still references emails/MyEmail.tsx. Either the prose descriptions should be updated to match the new filename, or the code block path should remain as MyEmail.tsx.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/docs/getting-started/monorepo-setup/npm.mdx, line 49:
<comment>Documentation inconsistency: The code block path was changed to `email.tsx` but the prose in Step 4 still instructs users to create `MyEmail.tsx`, and Step 6 still references `emails/MyEmail.tsx`. Either the prose descriptions should be updated to match the new filename, or the code block path should remain as `MyEmail.tsx`.</comment>
<file context>
@@ -46,24 +46,25 @@ Include the following script in your `package.json` file.
-```jsx packages/transactional/emails/MyEmail.tsx
-import { Button, Html } from "@react-email/components";
+```jsx packages/transactional/emails/email.tsx
+import { Button, Html, Head, Body } from "@react-email/components";
import * as React from "react";
</file context>
Summary by cubic
Refreshes the getting-started guides to streamline setup, standardize examples, and add Bun support. Updates screenshots with dedicated images for automatic and manual setup.
Written for commit 4afe3d7. Summary will update automatically on new commits.