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: add sanitizers github action #89

Merged
merged 5 commits into from
Dec 19, 2024
Merged

Conversation

danctorres
Copy link
Collaborator

@danctorres danctorres commented Dec 18, 2024

Description

Add job to run ASAN, UBSAN and TSAN on affected targets by changes on cpp targets, excluding test targets.

Checklist

  • Code follows project style guidelines.
  • Documentation updated (if applicable).
  • Tested in simulation/real-world environment (if applicable).

Testing

Tested by running github actions in CI.

Related Issue

Issue: #42

@danctorres danctorres added the wip Work in progress label Dec 18, 2024
@danctorres danctorres force-pushed the features/dt_add_sanitizers_action branch 18 times, most recently from f74bbc0 to d90072c Compare December 19, 2024 15:28
@danctorres danctorres force-pushed the features/dt_add_sanitizers_action branch from d90072c to e1c0fa2 Compare December 19, 2024 15:35
@danctorres danctorres force-pushed the features/dt_add_sanitizers_action branch from cb52070 to 6944d3a Compare December 19, 2024 15:50
@danctorres danctorres removed the wip Work in progress label Dec 19, 2024
@danctorres danctorres marked this pull request as ready for review December 19, 2024 15:54
@danctorres danctorres added the please review Make it obvious that PR requires review label Dec 19, 2024
@danctorres danctorres removed the please review Make it obvious that PR requires review label Dec 19, 2024
@danctorres danctorres merged commit 4706ce1 into main Dec 19, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants