-
Notifications
You must be signed in to change notification settings - Fork 2
Review 1.0 #3
Copy link
Copy link
Open
Description
theaspect
opened on Nov 15, 2021
Issue body actions
- 1. Перенеси все что относится к тестовым заданиями в отдельную папку: код, скрипты, ридми. Основной код пусть будет в src/main/kotlin
- 2. Тестовые сценарии и тестовые данные должны быть описаны в плане. Тесты должны запускаться на CI
- 3. If-ы в main должны быть последовательными а не вложенными 6.v
- 4. Логику парсинга нужно вынести из Arguments 5.vi
- 5. RoleResource 5.vi
- 6. isResource можно написать сильно проще, в пару строк
- 7. SourseData можно сделать более компактным, ненужные переносы на новые строки 2.iii
- 8. база данных должна инжектиться в AuthenticationProvider в мейне 5.xiv
- 9. попробуй переписать isResourceAccess через ФВП
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels