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

Updates to CI: use just, update tracking, enforce clang-tidy #1459

Closed
tvami opened this issue Sep 17, 2024 · 0 comments · Fixed by #1462
Closed

Updates to CI: use just, update tracking, enforce clang-tidy #1459

tvami opened this issue Sep 17, 2024 · 0 comments · Fixed by #1462
Assignees
Labels
config github_actions Pull requests that update GitHub Actions code

Comments

@tvami
Copy link
Member

tvami commented Sep 17, 2024

Is your feature request related to a problem? Please describe.

  • the CI compilion still uses the old approach
  • we updated the tracking configs
  • after Static analysis meta-issue #1176 is resolved, we should enforce those settings for the future code

Describe the solution you'd like

  • move CI compile to just
  • propagate new tracking config to the CI config too
  • enforce clang-tidy at CI level

Describe alternatives you've considered

We could discuss the last point at a software meeting, alternative seems more complicated when we do compile log diffs, in the end the developer should fix those problems

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config github_actions Pull requests that update GitHub Actions code
Projects
None yet
1 participant