Hw1#1
Conversation
|
@ITesserakt @Neytrinoo нет вердикта ни тут, ни на портале... |
|
Последний коммит, который был отправлен до дедлайна - bdd7d78 |
|
Не добавили меня в ревьюеры и не выставили в Assignee (на портале в п. 7 явно указано, что такие работы не проверяются!) |
leshiy1295
left a comment
leshiy1295
left a comment
There was a problem hiding this comment.
Сейчас искусственно ограничивается размер входных данных и нарушается SRP во многих местах. По CI - настроено довольно мало, проходит оно с замечаниями (нет Wall Wextra Wpedantic Werror-флагов). Тесты сейчас не изолированы и зависят через файловую систему друг от друга.
Условие отличается от того, которое есть на портале.
Даже если учитывать то, которое реализовано - с точки зрения "внешнего использования" она, может, и работает, но ДЗ всё же нацелено было на безопасное программирование и на проектирование базовой архитектуры, а с этими вещами получилось не очень...
| rm -rf build | ||
| mkdir build | ||
| cd build | ||
| cmake .. |
There was a problem hiding this comment.
сейчас в сборке большое количество замечаний, при этом сама сборка зелёная - сразу два недочёта
| - name: LINTER | ||
| run: | | ||
| cd Углубленный_C_C++/Homework_1/ | ||
| make check No newline at end of file |
There was a problem hiding this comment.
сейчас выдаётся замечание
nofile:0:0: information: Cppcheck cannot find all the include files (use --check-config for details) [missingInclude]
оно говорит о том, что не все include-ы Вы корректно написали
| sudo apt install lcov | ||
| cd build/ | ||
| lcov -t "tests/test_funcs" -o coverage.info -c -d funcs/ | ||
| genhtml -o report coverage.info |
There was a problem hiding this comment.
артефактов нет, поэтому посмотреть отчёт нельзя
| cmake --build . | ||
| cd .. | ||
| build/tests/test_fib | ||
|
|
There was a problem hiding this comment.
отсутствуют valgrind, санитайзеры и некоторые анализаторы (как минимум, fbinfer, scan-build)
There was a problem hiding this comment.
этих файлов не должно быть в репозитории
There was a problem hiding this comment.
в C++ принято использовать camelCase
There was a problem hiding this comment.
что эти функции тут делают? (как и pdf-файлы...)
There was a problem hiding this comment.
почему все тесты в одном файле?
2 participants
No description provided.