Skip to content

Remove clang-3 from CI#520

Merged
sbSteveK merged 1 commit intomainfrom
remove-clang-3
May 16, 2025
Merged

Remove clang-3 from CI#520
sbSteveK merged 1 commit intomainfrom
remove-clang-3

Conversation

@sbSteveK
Copy link
Contributor

@sbSteveK sbSteveK commented May 16, 2025

Remove clang-3 from CI

CI uses aws-crt-builder, which when instructed to use clang-3, has failed to find it and has been falling back on GNU 7.5 for the past year+. This means while clang-3 is listed in our CI as having been tested, it isn't. Fixing aws-crt-builder to use clang-3 has revealed a large number of build failures across aws-c-* libraries. Lack of issues related to this implies that clang-3 is not in use and can be removed from CI.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.72%. Comparing base (fbd8a91) to head (3060798).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #520   +/-   ##
=======================================
  Coverage   89.72%   89.72%           
=======================================
  Files          20       20           
  Lines        6382     6382           
=======================================
  Hits         5726     5726           
  Misses        656      656           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sbSteveK sbSteveK merged commit 50c9a33 into main May 16, 2025
34 checks passed
@sbSteveK sbSteveK deleted the remove-clang-3 branch May 16, 2025 18:01
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.

3 participants