Skip to content

Conversation

@mborland
Copy link
Member

@anarthal Can you please see if this change breaks your alpine environment? I added the CheckCXXSourceRuns to fix that environment but now apparently this breaks cross compilation: #193 (comment)

@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.84%. Comparing base (0c8358a) to head (b024cfc).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #284      +/-   ##
===========================================
- Coverage    94.92%   94.84%   -0.08%     
===========================================
  Files           73       73              
  Lines         9186     8986     -200     
===========================================
- Hits          8720     8523     -197     
+ Misses         466      463       -3     

see 47 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c8358a...b024cfc. Read the comment docs.

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

anarthal added a commit to anarthal/servertech-chat that referenced this pull request Oct 29, 2025
@anarthal
Copy link
Contributor

@mborland
Copy link
Member Author

Seems to be fine: https://github.com/anarthal/servertech-chat/actions/runs/18907969430/job/53970789363

Cool. Thanks for checking.

@mborland mborland merged commit e527477 into develop Oct 29, 2025
64 of 65 checks passed
@mborland mborland deleted the source_compiles branch October 29, 2025 13:38
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