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

Echo bot initial #1

Merged
merged 2 commits into from
Mar 12, 2024
Merged

Echo bot initial #1

merged 2 commits into from
Mar 12, 2024

Conversation

trollknurr
Copy link
Owner

Initial repo structure

Hexagonal architecture в полный рост

from tg_bot.container import Container
from domain.models import HumanTextInput

# TODO: Cool logging
Copy link
Collaborator

Choose a reason for hiding this comment

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

На всякий случай делюсь соской, довольно таки simple, но не слишком cool, потом может принесем с собой
https://github.com/Delgan/loguru

Copy link
Owner Author

Choose a reason for hiding this comment

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

Вынес в issues
#2

ENV PYTHONUNBUFFERED 1
ENV PYTHONWARNINGS once

RUN python -m pip install --upgrade pip poetry==1.8.2
Copy link
Collaborator

Choose a reason for hiding this comment

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

Прогнулся под молодежь, пересел на poetry, не ожидал...

Copy link
Owner Author

Choose a reason for hiding this comment

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

Я тот кто сажал молодежь на него

pyproject.toml Outdated
name = "hard-chill-bot"
version = "0.1.0"
description = ""
authors = ["Hard Chill Team <a.o.shtarev@gmail.com>"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Понятно, уже в одно жало решил приехать, узнаваемый стиль

Copy link
Owner Author

Choose a reason for hiding this comment

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

Стыдно/копипаста - исправил

pyproject.toml Outdated
src = ["src"]
line-length = 120
# Python 3.10.
target-version = "py310"
Copy link
Collaborator

Choose a reason for hiding this comment

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

В докере светится 3.11, чуть выше тоже , тут 3.10, возможно никак не сыграет, но можно поправить для единообразия.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Пофиксил копипасту

logger = logging.getLogger(__name__)


@inject
Copy link
Collaborator

Choose a reason for hiding this comment

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

Сексуально, почувствовал некоторое шевеление в штанах

Copy link
Owner Author

Choose a reason for hiding this comment

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

Инъектировал

pyproject.toml Outdated
[tool.black]
line-length = 120
# Python 3.10.
target-version = ["py310"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Снова 3.10, хотя например mypy настроен на 3.11, как и докер

Copy link
Owner Author

Choose a reason for hiding this comment

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

Пофиксил копипасту

@@ -0,0 +1,17 @@
format:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Один лайк и сварю pre-commit, чтобы случайно не уебаться на таких инициативах

Copy link
Collaborator

@evvfebruary evvfebruary left a comment

Choose a reason for hiding this comment

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

База подъехала

@trollknurr trollknurr merged commit 760fbf1 into main Mar 12, 2024
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