Skip to content

Conversation

@TsybulkaM
Copy link
Contributor

No description provided.

@TsybulkaM TsybulkaM added this to the Polish cities milestone Aug 29, 2025
@TsybulkaM TsybulkaM requested a review from Copilot August 29, 2025 12:03
@TsybulkaM TsybulkaM self-assigned this Aug 29, 2025
@TsybulkaM TsybulkaM linked an issue Aug 29, 2025 that may be closed by this pull request
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates the project from using python-dotenv to envsh for environment variable management, changing from .env files to .sh files for environment configuration.

Key changes:

  • Replace python-dotenv dependency with envsh library across all Python files
  • Update environment variable loading from load_dotenv() to envsh.load() with search paths
  • Replace os.getenv() calls with envsh.read_env() with type specification
  • Update Django settings module references to use development-specific settings

Reviewed Changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
pyproject.toml Remove python-dotenv dependency and add envsh, update Python version requirement
route_engine/route_engine/settings/base.py Replace dotenv with envsh for environment loading and variable reading
route_engine/route_engine/settings/prod.py Simplify ALLOWED_HOSTS configuration by replacing conditional logic with static list
route_engine/manage.py Update default Django settings module to development settings
logicway/routing/views.py Replace dotenv with envsh and remove Russian comment
logicway/map/views.py Replace dotenv with envsh for environment variable handling
logicway/logicway/settings/prod.py Simplify ALLOWED_HOSTS and remove CORS_ALLOWED_ORIGINS configuration
logicway/logicway/settings/base.py Replace dotenv with envsh and update database configuration
logicway/database/tests/models_database_test.py Update environment loading and add missing test data setup
logicway/database/tests/AJAX_test.py Replace dotenv with envsh and update test URLs to use environment variable
logicway/database/load_data.py Replace dotenv with envsh for database configuration
logicway/database/fetch_data_and_convert_to_json.py Replace dotenv with envsh for database configuration
logicway/database/database.py Replace dotenv with envsh for database configuration
frontend/package.json Update start script to source environment shell script
README.md Update documentation to reflect migration from .env to .sh files

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@TsybulkaM TsybulkaM merged commit 5c8db29 into main Aug 29, 2025
1 check passed
@TsybulkaM TsybulkaM removed this from the Polish cities milestone Aug 29, 2025
@TsybulkaM TsybulkaM deleted the 68-from-env-to-envsh-migration branch September 2, 2025 16:39
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.

from .env to envsh migration

2 participants