8000 run tests for nix by pca006132 · Pull Request #1094 · elalish/manifold · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

run tests for nix #1094

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8000
Merged
merged 2 commits into from
Dec 4, 2024
Merged

run tests for nix #1094

merged 2 commits into from
Dec 4, 2024

Conversation

pca006132
Copy link
Collaborator

No description provided.

Copy link
codecov bot commented Dec 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.87%. Comparing base (f1a8693) to head (b8aeb13).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1094   +/-   ##
=======================================
  Coverage   91.87%   91.87%           
=======================================
  Files          30       30           
  Lines        5883     5883           
=======================================
  Hits         5405     5405           
  Misses        478      478           

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

@@ -106,6 +106,7 @@
"-DBUILD_SHARED_LIBS=ON"
"-DMANIFOLD_PAR=${if parallel then "ON" else "OFF"}"
];
doCheck = true;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build_mxe still looks like it's not testing - also build_mac_..._subdir, unless that's intentional?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mxe cannot be tested: we are building some windows exe on linux. at least we cannot test it without another job on windows...

bulid_mac_..._subdir... is just to test the cmake consumer, see if we can build when put inside a subdirectory in a larger project.

@@ -106,6 +106,7 @@
"-DBUILD_SHARED_LIBS=ON"
"-DMANIFOLD_PAR=${if parallel then "ON" else "OFF"}"
];
doCheck = true;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And any idea why build_nix_tbb is so slow? It looks like it's building all of TBB before it even gets to Manifold...

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added the latest tbb version for emscripten, forgot why I also use it for other native builds... fixed.

Copy link
Owner
@elalish elalish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@pca006132 pca006132 merged commit 3603542 into master Dec 4, 2024
25 checks passed
@pca006132 pca006132 deleted the ci-fix branch December 4, 2024 01:07
@elalish elalish mentioned this pull request Dec 26, 2024
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.

2 participants
0