-
Notifications
You must be signed in to change notification settings - Fork 1
Migration from env to envsh #71
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.
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
Conversation
There was a problem hiding this 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-dotenvdependency withenvshlibrary across all Python files - Update environment variable loading from
load_dotenv()toenvsh.load()with search paths - Replace
os.getenv()calls withenvsh.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.
No description provided.