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.

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

Jump to bottom

feat: add environment variable configuration #33

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: add environment variable configuration #33

wants to merge 1 commit into from

Conversation

deepatidarr
Copy link

Issue Description:

  • Currently, the index.ts file contains a hardcoded localhost address
  • This can cause issues when configuring the project for different environments
  • A .env file should be introduced to allow flexible configuration

Changes Implemented:

  • Updated index.ts to read the localhost address from an .env file using dotenv
  • Added a .env_example file with the required environment variables
  • Updated TypeScript configuration and dependencies
  • Added proper type definitions

Steps to Test:

  1. Copy .env_example to .env
  2. Update values in .env as needed
  3. Run npm install
  4. Start the server and verify it reads from .env file

feat: add environment variable configuration
aa3144f 
- Replaced hardcoded localhost address with environment variables
- Added .env_example for reference
- Updated TypeScript configuration
- Added proper type definitions

Signed-off-by: Deep Patidar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@deepatidarr