Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

TSAN CI changes #555

Merged
merged 2 commits into from
Sep 8, 2022
Merged

TSAN CI changes #555

merged 2 commits into from
Sep 8, 2022

Conversation

mjp41
Copy link
Member

@mjp41 mjp41 commented Sep 8, 2022

  • Remove TSAN Debug - it takes too long, and it is not clear it adds any additional checking over TSAN Release build
  • Require TSAN to pass for approval - TSAN this was not initially enabled as there were issues, but it has been stable for a long time, so can be part of the approval process.

@mjp41 mjp41 merged commit 2ee522c into microsoft:main Sep 8, 2022
@mjp41 mjp41 deleted the tsan_debug branch September 8, 2022 10:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants