search

HeavyTechRuby / history

History of every building
GNU General Public License v3.0
8 stars 8 forks source link

Update code coverage badge from codeclimate #47

Closed sas2job closed 3 months ago

sergio-fry commented 3 months ago

@sas2job а можешь дать ссылку, где у тебя отображается coverage?

sas2job commented 3 months ago

@sergio-fry Лучше этот PR удалить.

sas2job commented 3 months ago

@sergio-fry image image

@sergio-fry На codecov репозиторий активирован?

sergio-fry commented 3 months ago

А как это проверить? Может, я env неверно выставил?

sergio-fry commented 3 months ago

У меня есть предложение сделать через codecliemate, который у нас уже тоже подключен. Чтобы не плодить сервисы и чтобы у него было больше информации о нашем коде, так будут более информативные отчеты https://docs.codeclimate.com/docs/configuring-test-coverage

sergio-fry commented 3 months ago

Собственно, текст ошибки такой

Error: No coverage reports found. Please make sure you're generating reports successfully. Warning: Codecov: Failed to properly upload report: The process

sas2job commented 3 months ago

в таком виде замерджить не моге, так как в этом репо точно не сработает. Понимаю, что отлаживался у себя, но в итогом PR нужно выставить корректное значение.

Именно так. Перепутал репозиторий для PR.

sas2job commented 3 months ago

Собственно, текст ошибки такой

Error: No coverage reports found. Please make sure you're generating reports successfully. Warning: Codecov: Failed to properly upload report: The process

Эта ошибка решается путем добавления гема который создаёт отчёт после запуска тестов, но раз переходим на codeclimate то буду настраивать его.

sergio-fry commented 3 months ago

@sas2job какое у нас следующее действие и на чьей стороне мяч?

sas2job commented 3 months ago

@sas2job какое у нас следующее действие и на чьей стороне мяч?

@sergio-fry На моей. Убираю гемы связанные с codecov. Настраиваю на форке работу с codeclimate. Как настрою у себя, скину скриншот сюда и буду делать на текущей ветке чтобы не множить сущности.

sas2job commented 3 months ago

@sergio-fry на локальной версии image

sas2job commented 3 months ago

@sergio-fry Нужен код из настроек бейджа после переезда репозитория. См картинку image Также необходимо обновить или добавить CC_TEST_REPORTER_ID После того как скинешь код бейджа продолжу дальше.

sergio-fry commented 3 months ago

CC_TEST_REPORTER_ID выставил

а вот код badge

[![Test Coverage](https://api.codeclimate.com/v1/badges/a3a5daaab0db5b759cc3/test_coverage)](https://codeclimate.com/github/HeavyTechRuby/history/test_coverage)
sergio-fry commented 3 months ago

@sas2job просьба к тебе, сделать заголовок PR таким, чтобы было понятно, о чем речь внутри

sas2job commented 3 months ago

@sergio-fry

Как правильно оформить дальнейшую работу? Ветка patch-3 с которой создан текущий PR создана через интерфейс github при внесении изменений в файл ci.yml Сейчас изменения находятся в ветке main форка репозитория.

sergio-fry commented 3 months ago

Думаю, тебе нужно перенести изменения в ту ветку, которая является рабочей для данного PR, то есть sas2job:sas2job-patch-3

codeclimate[bot] commented 3 months ago

Code Climate has analyzed commit 9e31268d and detected 0 issues on this pull request.

View more on Code Climate.

sergio-fry commented 3 months ago

что-то он не подхватывает CC_TEST_REPORTER_ID, хотя я добавил его в секерты. Подскажи, а как ты это настраивал?

sas2job commented 3 months ago

что-то он не подхватывает CC_TEST_REPORTER_ID, хотя я добавил его в секерты. Подскажи, а как ты это настраивал?

В своём репозиториивот так, добавлял через веб интерфейс image

sergio-fry commented 3 months ago

так я добавил уже )

Screenshot 2024-06-05 at 12 20 28

Давай замерджим, может в мастере сработает

sergio-fry commented 3 months ago

Так и есть - ENV доступна только в мастере. Короче, сделаю ее менее секретной, иначе у нас в PR будут сборки падать

sergio-fry commented 3 months ago

Или нет. Нужно сделать так, чтобы этот job запускался только в main. Создал #74