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.

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

Simple keira class generator #79

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

frostmorn
Copy link
Collaborator

@frostmorn frostmorn commented Mar 28, 2024

Usage chmod +x keira_gen.py
./keira_gen.py app/service name
Could be moved to utils, or somewhere else,
for this purpose could be used with KEIRA_TEMPLATES_PATH and KEIRA_SOURCE_PATH env vars

@and3rson and3rson force-pushed the main branch 2 times, most recently from e17861d to d9904e9 Compare March 28, 2024 16:12
@and3rson
Copy link
Owner

and3rson commented Mar 30, 2024

Хороша ідея. Добре було б це винести з src в іншиу директорію - скажімо, tools

@and3rson
Copy link
Owner

Можна задати якісь дефолтові значення для env-змінних, наприклад:

this_dir = Path(__file__).parent
os.environ.setdefault("KEIRA_SOURCE_PATH", str((this_dir / ".." / ".." / "src").resolve()))
os.environ.setdefault("KEIRA_TEMPLATES_PATH", str(this_dir / "templates"))

@frostmorn
Copy link
Collaborator Author

frostmorn commented Apr 1, 2024

Можна задати якісь дефолтові значення для env-змінних

це вже було насправді в попередній версії, але вирішив прибрати так як ми не можемо знати з якої саме директорії користувач захоче використовувати ці скрипти. тому так, аби бути впевненим в тому що відповідні змінні середовища задані

@and3rson
Copy link
Owner

and3rson commented Apr 4, 2024

Можна задати якісь дефолтові значення для env-змінних

це вже було насправді в попередній версії, але вирішив прибрати так як ми не можемо знати з якої саме директорії користувач захоче використовувати ці скрипти. тому так, аби бути впевненим в тому що відповідні змінні середовища задані

Але ми знаємо, в якій директорії знаходиться скрипт, тому можемо взяти дефолтовий шлях відносно файлу скрипта (__file__), який завжди буде містити шлях до скрипта, і за замовчуванням писати в src/apps відносно директорії, в якій лежить __file__.
Основна причина, чому мені це здається необхідним - це час: скоріш за все, юзеру піде більше часу, щоб правильно прописати env-змінні для запуску скрипта, ніж скопіювати бойлерплейт-файли вручну, що зруйнує ціль скрипта :)

@frostmorn
Copy link
Collaborator Author

frostmorn commented Apr 4, 2024

Основна причина, чому мені це здається необхідним - це час: скоріш за все, юзеру піде більше часу, щоб правильно прописати env-змінні для запуску скрипта, ніж скопіювати бойлерплейт-файли вручну, що зруйнує ціль скрипта :)

якщо це займає настільки багато часу для юзера, то можливо не зовсім потрібно аби він писав якийсь код,
зрештою підхід який ти пропонуєш він звісно непоганий, однак.

я взагалі пропоную зробити якусь таку папку десь в lilka/tools, і позапихувати туди усі подібні штуки, ввести якусь змінну середовища по типу LILKA_PROJECT_ROOT, і вже від цього щось думати.

тоді вже можна було б стандартизувати використання __file__

@and3rson and3rson force-pushed the main branch 2 times, most recently from 4563cc8 to 14a952c Compare April 17, 2024 18:14
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.

None yet

2 participants