Skip to content

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 79.61%. Comparing base (3eedf1e) to head (7e66f12).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #518   +/-   ##
=======================================
  Coverage   79.61%   79.61%           
=======================================
  Files          27       27           
  Lines       11702    11702           
=======================================
  Hits         9316     9316           
  Misses       2386     2386           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sbSteveK sbSteveK merged commit fcc4932 into main May 16, 2025
41 checks passed
@sbSteveK sbSteveK deleted the remove-clang-3 branch May 16, 2025 18:01
quinnj pushed a commit to quinnj/aws-c-http that referenced this pull request Oct 25, 2025
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