Skip to content

make test_rust will automatically install cargo-llvm-cov if it's not found#1099

Merged
400Ping merged 1 commit intoapache:mainfrom
ryankert01:makefile-more-robust
Feb 28, 2026
Merged

make test_rust will automatically install cargo-llvm-cov if it's not found#1099
400Ping merged 1 commit intoapache:mainfrom
ryankert01:makefile-more-robust

Conversation

@ryankert01
Copy link
Member

@ryankert01 ryankert01 commented Feb 27, 2026

Related Issues

Closes #1097

Changes

  • Bug fix
  • New feature
  • Refactoring
  • Documentation
  • Test
  • CI/CD pipeline
  • Other

Why

How

make sure cargo-llvm-cov is installed in makefile to make it more robust

Checklist

  • Added or updated unit tests for all changes
  • Added or updated documentation for all changes

Copy link
Contributor

@viiccwen viiccwen left a comment

Choose a reason for hiding this comment

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

yeah, It's missing.
thx for the patch!

@400Ping 400Ping changed the title make test_rust will automatically install cargo-llvm-cov if it's not … make test_rust will automatically install cargo-llvm-cov if it's not found Feb 28, 2026
@400Ping 400Ping merged commit 5db5e8d into apache:main Feb 28, 2026
2 checks passed
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.

[Bug] Makefile make test not robust

3 participants