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

CI integration #154

Merged
merged 25 commits into from
May 15, 2024
Merged

CI integration #154

merged 25 commits into from
May 15, 2024

Conversation

apmilko
Copy link
Member

@apmilko apmilko commented May 13, 2024

Интеграция CI, сделано полностью как в репозитории handy, но с парой уточнений.

Основные этапы:

  • [DONE] Джоба для запуска пре-коммита
  • [TODO] Джоба для сборки проекта и запуска тестов

Джоба для запуска пре-коммита:

  1. Исправлена ошибка в хуке clang-format (types_or вместо types), из-за которой множество файлов для запуска проверки определялось не корректно
  2. Убрал в конце ссылки на репозиторий для хука yamlint лишнюю преписку .git
  3. К хуку check-added-large-files добавлены 2 аргумента: --enforce-all (для проверки всех файлов в репозитории), --maxkb=12300 (см. комментарий)
  4. В конфигурации хука .yamllint-config.yaml максимальная длина строки изменена с 120 до 180 (см. комментарий)
  5. Все хуки проходят проверку и возвращают Passed
  6. После применения хуков проект собирается (с применением временного фикса для работы библиотеки eigen3) [эта бага не относится к CI и была до него]

Signed-off-by: Andrey Milko <[email protected]>
@apmilko apmilko self-assigned this May 13, 2024
@apmilko apmilko requested review from dasimagin and AndBondStyle May 13, 2024 14:19
@AndBondStyle
Copy link
Member

По поводу бага с eigen3 - надо поправить в мастере

apmilko added 3 commits May 15, 2024 15:20
Config is default except one param 'dangle_parens = True',
which is used by next rule: if a statement is wrapped
to more than one line, than dangle the closing parenthesis
on its own line.

Exclude file ".cmake-format.py" for hook "ruff".
BreakBeforeBinaryOperators: NonAssignment
BreakConstructorInitializers: AfterColon
@apmilko apmilko merged commit 1accbe1 into master May 15, 2024
1 check passed
@apmilko apmilko deleted the apmilko-ci branch May 15, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants