Skip to content

Conversation

@yamz8
Copy link

@yamz8 yamz8 commented Nov 17, 2025

make it follow the file in the drizzle.config.ts

make it follow the file in the drizzle.config.ts
@vercel
Copy link
Contributor

vercel bot commented Nov 17, 2025

@yamz8 is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

cd coding-agent-template
pnpm install
# Set up .env.local with required variables
# Set up .env with required variables
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Set up .env with required variables
# Set up .env.local with required variables

The Quick Start section now references .env for environment variables, but the detailed "Local Development Setup" section still references .env.local. This inconsistency will confuse developers during setup.

View Details

Analysis

Documentation inconsistency: Quick Start references wrong environment file

What fails: README.md Quick Start section (line 46) instructs developers to "Set up .env with required variables" while the detailed "Local Development Setup" section (line 188) instructs to "Create a .env.local file with your values". This creates conflicting setup instructions for local development.

How to reproduce:

  1. Read line 46 in Quick Start - says to use .env
  2. Read line 188 in "Local Development Setup" - says to use .env.local
  3. Developer follows Quick Start and creates .env, then encounters different instructions in detailed setup

Result: Developers are confused about which file to use since Quick Start contradicts the detailed setup section.

Expected: Per Next.js environment variables documentation, .env.local is the standard convention for local development - it allows developers to override defaults without committing sensitive values to version control. The gitignore pattern .env* is specifically designed to ignore .env.local for local development.

Fix: Updated line 46 to reference .env.local to align with line 188 and follow Next.js best practices.

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