Skip to content

Install endian header on Windows and Apple only - #372

Merged
urfeex merged 1 commit into
UniversalRobots:masterfrom
urfeex:fix_endian_header_installation
Sep 15, 2025
Merged

Install endian header on Windows and Apple only#372
urfeex merged 1 commit into
UniversalRobots:masterfrom
urfeex:fix_endian_header_installation

Conversation

@urfeex

@urfeex urfeex commented Sep 3, 2025

Copy link
Copy Markdown
Member

Installing it also on Linux may lead to the compiler finding the wrong implementation.

@urfeex
urfeex requested review from a team and URJala September 3, 2025 09:36
@urfeex urfeex added the bugfix label Sep 3, 2025
@codecov

codecov Bot commented Sep 3, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.52%. Comparing base (7967284) to head (d5b2efc).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #372      +/-   ##
==========================================
- Coverage   78.10%   77.52%   -0.58%     
==========================================
  Files          92       92              
  Lines        4161     4156       -5     
  Branches      461      461              
==========================================
- Hits         3250     3222      -28     
- Misses        675      695      +20     
- Partials      236      239       +3     
Flag Coverage Δ
start_ursim 81.67% <ø> (-2.54%) ⬇️
ur20-latest 75.55% <ø> (-0.29%) ⬇️
ur5-3.14.3 76.10% <ø> (-0.24%) ⬇️
ur5e-10.7.0 70.74% <ø> (-0.40%) ⬇️
ur5e-5.9.4 76.06% <ø> (-0.13%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@urfeex
urfeex merged commit 45404e8 into UniversalRobots:master Sep 15, 2025
23 of 38 checks passed
@urfeex
urfeex deleted the fix_endian_header_installation branch September 15, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants