Skip to content

Conversation

kushalhalder
Copy link

No description provided.

Copy link

@jondmarien jondmarien left a comment

Choose a reason for hiding this comment

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

This change to include the .env file path in langgraph.json looks good and should help resolve configuration issues like those mentioned in #11. The relative path './.env' is appropriate for the project structure.

However, since this changes how configuration is loaded, it would be helpful to:

  1. Update the README to document this change
  2. Consider adding a .env.example file to show required variables

Otherwise, the change itself looks correct and is a good improvement.

Copy link

@jondmarien jondmarien left a comment

Choose a reason for hiding this comment

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

The changes look good and will improve the configuration experience. The addition of the env path in langgraph.json is a sensible solution for handling environment variables.

Once the documentation updates are added as suggested in the previous comment, this will be ready to merge.

@mdrxy
Copy link
Contributor

mdrxy commented Jul 10, 2025

@kushalhalder please provide a description

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.

3 participants