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

Checkpoint 2 #2

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

Checkpoint 2 #2

wants to merge 9 commits into from

Conversation

VPErmakov
Copy link
Contributor

No description provided.

@VPErmakov VPErmakov requested a review from linlit April 10, 2022 18:07
@linlit
Copy link
Collaborator

linlit commented Apr 29, 2022

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

давайте поступим так, вы мне нарисуете архитектуру спринг бута с полными развернутыми комментариями, обоснуете зачем он нужен вам в проекте по ТП, как он устроен - тогда зачту как-либо задание. примерно таким же образом вы должны обосновать использование dependecy injection-ов в вашем случае. почему вот вы написали, что их помощью именно связность классов уменьшите? неужели мы за курс не выяснили других подходов?

@VPErmakov
Copy link
Contributor Author

Извини, мне казалось, что первый чекпоинт мы залили, сейчас исправим. По-поводу спринг бута и архитектуры готовим ответ

@linlit
Copy link
Collaborator

linlit commented May 15, 2022

окей, подготовьте поскорее, пожалуйста. пока поставлю за 2 и 3 чекпоинты по 5 обоим, чтобы что-то хотя бы стояло. но обидно вам пятерки ставить

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.

2 participants