Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Alexandwere created a review comment on a pull request on no-fedov/city
Можно добавить что-то, если условие не выполнится. Иначе будет непонятно взял работник задачу или нет
Alexandwere created a review comment on a pull request on no-fedov/city
Почему не public? Из другого класса к методу нет доступа.
Alexandwere created a review comment on a pull request on no-fedov/city
Добавить текст в ошибку, иначе не понятна причина
Alexandwere created a review comment on a pull request on no-fedov/city
Можно добавить аннотацию @FieldDefaults(level = AccessLevel.PRIVATE) И убрать private у свойств класса
Alexandwere created a review comment on a pull request on no-fedov/city
NonNull перед BigDecimal
Alexandwere created a review comment on a pull request on no-fedov/city
А здесь false.
Alexandwere created a review comment on a pull request on no-fedov/city
Здесь нужен true, Иначе наоборот ставятся все ставки, кроме 1500-2000
no-fedov pushed 2 commits to stage3_tasks no-fedov/city
no-fedov pushed 14 commits to stage3_tasks no-fedov/city
- feat: формирование pom-файла b70df0c
- feat: Частично заполнен pom, реализован Human 434329a
- стянул изменени 5ef5ff1
- стянул к себе ветки 6a381c4
- Стабильная версия 23bc3e1
- Стабильная версия 2 f1bd37e
- добавли загс 07e6c41
- стянул 61746c7
- регитсрация ребенка начало 218a243
- регитсрация ребенка стянул таск 3 0011508
- обновил ветку ccd90bd
- регитсрация ребенка стянул таск обновил 487bf55
- feat: реализован метод регистрации ребенка, общая логика вынесена в отдельные методы 225ca66
- feat: полностью реализована логика ЗАГСа 0558360
Aslamkhanov pushed 15 commits to civil_registry_anbu no-fedov/city
- feat: организован root pom, сделан модуль human (#2) 060aca3
- feat: разработка модуля загс начата 589990d
- fix: размечена работа для сущносстей модуля 3, расписаны задачи d19e796
- fix: заменить Date на LocalDate f34ebbc
- style: убрать неиспользуемый импорт 26c4d8a
- fix: убрать Setter из CivilActionRecord 8f6983b
- добавли загс 07e6c41
- feat: реализованы методы регистрации брака и развода df0db5f
- стянул 61746c7
- регитсрация ребенка начало 218a243
- refactor: изменена сигнатура private метода makeCivilActionRecord 66a800f
- регитсрация ребенка стянул таск 3 0011508
- обновил ветку ccd90bd
- регитсрация ребенка стянул таск обновил 487bf55
- добавил папки и регистрацию рождения ребенка 5d6b2db
karina7575 created a review comment on a pull request on no-fedov/city
добавила проверку задачи на решенность в начале метода
no-fedov pushed 1 commit to stage3_tasks no-fedov/city
- refactor: изменена сигнатура private метода makeCivilActionRecord 66a800f
no-fedov created a review comment on a pull request on no-fedov/city
isSolved, isOver больше подходит
no-fedov created a review comment on a pull request on no-fedov/city
Программист может получить деньги за решенную задачу, этот лучше учесть, чтобы просто так ему не платить
no-fedov created a review comment on a pull request on no-fedov/city
Название параметра rateToCheck лучше заменить на rate, это же сеттер Проверка есть, но значение не установлено. Также магические числа 1500 2000 вынести в константы. (По желанию) Можно подума...
no-fedov pushed 1 commit to stage3_tasks no-fedov/city
- feat: реализованы методы регистрации брака и развода df0db5f
karina7575 pushed 1 commit to human_professions no-fedov/city
- feat: реализована логика получения задачи программистом 5505c63