Skip to content

fix(README): Resolve markdownlint issues #469

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.

Sign up for GitHub

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

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

fix(README): Resolve markdownlint issues #469

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
85a771a
fix(README): Resolve markdownlint issues
hvert Aug 16, 2025
27e3857
fix(README): Standardize URL formatting and correct code block syntax
hvert Aug 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ A production-ready example of a multi-tenant application built with Next.js 15,
3. Set up environment variables:
Create a `.env.local` file in the root directory with:

```
```bash
KV_REST_API_URL=your_redis_url
KV_REST_API_TOKEN=your_redis_token
```
Expand All @@ -59,9 +59,9 @@ A production-ready example of a multi-tenant application built with Next.js 15,
```

5. Access the application:
- Main site: http://localhost:3000
- Admin panel: http://localhost:3000/admin
- Tenants: http://[tenant-name].localhost:3000
- Main site: <http://localhost:3000>
Copy link

@vercel vercel bot Aug 16, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent URL formatting where two URLs use markdown link format but the third remains in plain text.

View Details
📝 Patch Details 
diff --git a/README.md b/README.md
index 178364c..ca39f97 100644
--- a/README.md
+++ b/README.md
@@ -61,7 +61,7 @@ A production-ready example of a multi-tenant application built with Next.js 15,
 5. Access the application:
    - Main site: <http://localhost:3000>
    - Admin panel: <http://localhost:3000/admin>
-   - Tenants: http://[tenant-name].localhost:3000
+   - Tenants: <http://[tenant-name].localhost:3000>
 
 ## Multi-Tenant Architecture

Analysis

The URLs in the "Access the application" section have inconsistent formatting. Lines 62-63 use markdown angle bracket format (<http://...>), while line 64 uses plain text format (http://...). This inconsistency makes the documentation appear unpolished and may confuse readers about which URLs are clickable links.

In markdown viewers, the angle bracket format creates clickable links, while plain text URLs may not be automatically converted to links depending on the viewer.

Recommendation

Make all URLs consistent by either:

  1. Using angle brackets for all URLs:
- Main site: <http://localhost:3000>
- Admin panel: <http://localhost:3000/admin>
- Tenants: <http://[tenant-name].localhost:3000>
  1. Or using plain text for all URLs:
- Main site: http://localhost:3000
- Admin panel: http://localhost:3000/admin
- Tenants: http://[tenant-name].localhost:3000

The angle bracket format is recommended as it ensures the URLs are clickable links in markdown viewers.

- Admin panel: <http://localhost:3000/admin>
- Tenants: <http://[tenant-name].localhost:3000>

## Multi-Tenant Architecture

Expand Down