Skip to content

Conversation

@aaronj0
Copy link
Collaborator

@aaronj0 aaronj0 commented May 6, 2024

  • reverts Python version to 3.10
  • Disables specific CppInterOp unit tests causing bad valgrind reports
  • Added flag for CUDA test to keep disabled during the runs

@aaronj0 aaronj0 mentioned this pull request May 6, 2024
@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2024

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2024

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2024

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2024

clang-tidy review says "All clean, LGTM! 👍"

@codecov
Copy link

codecov bot commented May 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.45%. Comparing base (f93e7b6) to head (ec97acb).
Report is 17 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #268       +/-   ##
===========================================
- Coverage   87.65%   75.45%   -12.21%     
===========================================
  Files           4        8        +4     
  Lines        1742     2994     +1252     
===========================================
+ Hits         1527     2259      +732     
- Misses        215      735      +520     

see 8 files with indirect coverage changes

see 8 files with indirect coverage changes

@mcbarton
Copy link
Collaborator

mcbarton commented May 6, 2024

@maximusron can you add to this PR commits which disable the failing tests on Windows, so we can remove the continue on error for that part of the ci?

@vgvassilev vgvassilev merged commit bed3b05 into compiler-research:main May 7, 2024
@aaronj0 aaronj0 deleted the disable-tests branch April 22, 2025 08:25
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