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

[MOD] Формат Code Qualty GitLab #3419

Open
spawnax opened this issue Feb 14, 2025 · 12 comments · May be fixed by #3424
Open

[MOD] Формат Code Qualty GitLab #3419

spawnax opened this issue Feb 14, 2025 · 12 comments · May be fixed by #3424
Labels

Comments

@spawnax
Copy link

spawnax commented Feb 14, 2025

Диагностика

Проверка файлов по списку

Описание предлагаемых изменений

Необходимо добавить формат отчета для прямого использования в GitLAB
Вот ссылка на формат, он простой похож на tslint
https://docs.gitlab.com/ee/ci/testing/code_quality.html#code-quality-report-format

Дополнительная информация

@spawnax spawnax added component/diagnostics Доработка / создание диагностики type/discuss labels Feb 14, 2025
@nixel2007
Copy link
Member

В использовании оно будет чем-то отличаться от junit? Junit бсл лс поддерживается в гитлабе.

@nixel2007 nixel2007 added type/enhancement New feature or request component/reporter and removed type/discuss component/diagnostics Доработка / создание диагностики labels Feb 14, 2025
@spawnax
Copy link
Author

spawnax commented Feb 15, 2025

Да, в гитлабе это разные сущности, юнит-тесты выдают сводную инфу в своём дашборде:
https://docs.gitlab.com/ee/ci/testing/img/junit_test_report_v13_9.png

А проверка кода всё же ближе к нашим целям, она отображает все замечания прямо в коммитах и мёрджреквестах:
https://docs.gitlab.com/ee/ci/testing/img/code_quality_inline_indicator_v16_7.png

@nixel2007
Copy link
Member

Понял, спасибо.
Принято!

@nixel2007
Copy link
Member

А сонар для этих целей (анализа мердж реквестов) использовать не планируете?

@spawnax
Copy link
Author

spawnax commented Feb 18, 2025

Сонар всё ж не нативный, а отдельная система со своим интерфейсом, которая требует отдельного организованного процесса проверки, а тут прям в рамках гитлаба удобно каждый каммит проверять, быстренько разраба "потыкать палочкой".

@nixel2007
Copy link
Member

Так сонар умеет в гитлабе построчно комментировать мерж реквесты. Даже в бесплатной версии. Интерфейс сонара не обязательно открывать.

Или code quality умеет просто в покоммитный просмотр из основной ветки?

@spawnax
Copy link
Author

spawnax commented Feb 19, 2025

Да, code quality каждый каммит прямо в при просмотре его изменений сразу отображать замечания. Т.е. грубо говоря, выложил у меня разраб коммит, отработал runner, увидел косяки, я ему сразу на пайплайн вешаю отказ, а не ждём пока задачу разраб закончит, и только разборе мердж-реквеста мы ему сделаем ататай.

@nixel2007
Copy link
Member

Здорово, звучит очень полезно.
Покоммитный анализ в бранчах в сонаре тоже есть, но поинт про необходимость перехода во внешнюю систему вполне валидный, да.

nixel2007 added a commit that referenced this issue Feb 20, 2025
@nixel2007 nixel2007 linked a pull request Feb 20, 2025 that will close this issue
5 tasks
@nixel2007
Copy link
Member

nixel2007 commented Feb 20, 2025

@spawnax попробуйте, пожалуйста, сборку вот отсюда: #3424

./gradlew bootJar

Искать в каталоге build/libs

Ключ репортера - code-quality

nixel2007 added a commit that referenced this issue Feb 20, 2025
@nixel2007
Copy link
Member

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

@spawnax
Copy link
Author

spawnax commented Feb 20, 2025

Могу я вас ещё попросить скомпилировать под win_x64, я сам скорее всего быстро не осилю

@sfaqer
Copy link
Member

sfaqer commented Feb 20, 2025

Могу я вас ещё попросить скомпилировать под win_x64, я сам скорее всего быстро не осилю

https://dropmefiles.com/iNE23

(github не даёт приложить жарник\рарник\зипарь запороленный, так что ссылка.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants