Skip to content

сдача задания по второму спринту - #1

Open
ppr51 wants to merge 3 commits into
mainfrom
dev
Open

ppr51 wants to merge 3 commits into
mainfrom
dev

Conversation

@ppr51

@ppr51 ppr51 commented Mar 28, 2023

Copy link
Copy Markdown
Owner

от Полховского Павла
7 когорта по Андроид разработке

Comment thread src/main/java/Main.java Outdated
Comment thread src/main/java/Main.java Outdated
Comment thread src/main/java/Main.java Outdated
Comment thread src/main/java/Main.java Outdated
Comment thread src/main/java/BillCalculator.java Outdated
Comment thread src/main/java/BillCalculator.java Outdated
Comment thread src/main/java/PriceFormatter.java Outdated
@SeveNChaK

Copy link
Copy Markdown

В некоторых местах отступы не одинаковые - рекомендую использовать встроенные форматтер. Если ты программируешь в IDE от JetBrains, то можешь пройтись по всем файлам и нажать комбинацию (Windows: Shift + Ctrl + Alt + L, MacOS: Shift + Option + Command + L), затем выбрать, какую часть кода хочешь изменить, и выполнить команду. Среда разработки автоматически сделает нужные отступы и переносы (как она это делает можно изменить в настройках, но дефолтные значения вполне нормальные).

@ppr51

ppr51 commented Mar 30, 2023

Copy link
Copy Markdown
Owner Author

не знаю нужен ли новый пулл реквест или текущего достаточно, оставил текущий

@SeveNChaK

SeveNChaK commented Mar 31, 2023 •

Copy link
Copy Markdown

не знаю нужен ли новый пулл реквест или текущего достаточно, оставил текущий

лучше старый ПР оставлять для исправлений. Но как вариант, можно делать новую ветку с исправлениями и делать новый ПР между ветками первой итерации и итерацией с исправлениями

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