pistacheio / pistache

A high-performance REST toolkit written in C++
https://pistacheio.github.io/pistache/
Apache License 2.0
3.12k stars 690 forks source link

ci(linux): use LLD when using Clang #1092

Closed Tachi107 closed 1 year ago

Tachi107 commented 1 year ago

This allows us to test that Pistache links and works correctly with LLVM's LLD linker, and also fixes the issues with the RHEL 8 / Clang job

codecov-commenter commented 1 year ago

Codecov Report

Merging #1092 (cfb3843) into master (5bf1a89) will decrease coverage by 0.18%. The diff coverage is n/a.

:exclamation: Current head cfb3843 differs from pull request most recent head 0a74ab1. Consider uploading reports for the commit 0a74ab1 to get more accurate results

@@            Coverage Diff             @@
##           master    #1092      +/-   ##
==========================================
- Coverage   78.30%   78.11%   -0.19%     
==========================================
  Files          53       52       -1     
  Lines        6657     6531     -126     
==========================================
- Hits         5213     5102     -111     
+ Misses       1444     1429      -15     
Impacted Files Coverage Δ
include/pistache/router.h 94.23% <0.00%> (-5.77%) :arrow_down:
src/client/client.cc 82.00% <0.00%> (-2.38%) :arrow_down:
src/common/tcp.cc 81.81% <0.00%> (-1.52%) :arrow_down:
src/common/peer.cc 76.05% <0.00%> (-1.41%) :arrow_down:
src/common/stream.cc 83.33% <0.00%> (-1.21%) :arrow_down:
src/common/os.cc 59.53% <0.00%> (-0.58%) :arrow_down:
include/pistache/net.h 84.61% <0.00%> (-0.57%) :arrow_down:
include/pistache/http_headers.h 80.00% <0.00%> (-0.56%) :arrow_down:
include/pistache/http.h 86.11% <0.00%> (-0.38%) :arrow_down:
include/pistache/endpoint.h 94.73% <0.00%> (-0.27%) :arrow_down:
... and 22 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.