Skip to content

Conversation

@yungwine
Copy link
Collaborator

@yungwine yungwine commented Oct 1, 2025

No description provided.

@yungwine yungwine requested a review from Copilot October 1, 2025 10:12
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 refactors the installation script to support long-form command line arguments and improves environment variable handling. The changes move from direct environment variable access to a centralized configuration approach using a CONFIG dictionary.

  • Adds support for long-form command line arguments (e.g., --config, --mode) alongside existing short forms
  • Introduces environment file support via --env-file option
  • Refactors environment variable handling from direct os.environ access to a centralized CONFIG dictionary

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
scripts/install.sh Adds long-form argument parsing and environment file support
scripts/install.py Refactors environment variable handling to use CONFIG dictionary
mytoninstaller/settings.py Updates to use buffer attributes instead of direct environment variable access
mytoninstaller/mytoninstaller.py Centralizes environment variable initialization in init_envs function

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

@yungwine yungwine changed the title Args Add long installing options, add env file providing on install, add --print-env option Oct 1, 2025
@yungwine yungwine merged commit daab5bd into ton-blockchain:dev Oct 1, 2025
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