Skip to content

Conversation

@brianheineman
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 28, 2025

🐰 Bencher Report

Branchremove-value-to-methods
TestbedLinux

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds flag.

Click to view all benchmark results
BenchmarkLatencynanoseconds (ns)
default_class_loader📈 view plot
⚠️ NO THRESHOLD
6,986,400.00 ns
from_bytes📈 view plot
⚠️ NO THRESHOLD
31,952.00 ns
hello_world📈 view plot
⚠️ NO THRESHOLD
193,650,000.00 ns
load_hash_map📈 view plot
⚠️ NO THRESHOLD
478.50 ns
load_invalid_class📈 view plot
⚠️ NO THRESHOLD
29,216.00 ns
to_bytes📈 view plot
⚠️ NO THRESHOLD
17,773.00 ns
to_string📈 view plot
⚠️ NO THRESHOLD
125,830.00 ns
verify📈 view plot
⚠️ NO THRESHOLD
821.70 ns
vm_init📈 view plot
⚠️ NO THRESHOLD
178,510,000.00 ns
🐰 View full continuous benchmarking report in Bencher

@brianheineman brianheineman force-pushed the remove-value-to-methods branch 2 times, most recently from 200d034 to 8441636 Compare July 28, 2025 21:54
@codecov
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 86.59794% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.16%. Comparing base (1b7b779) to head (4063196).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...m/src/intrinsic_methods/java/lang/reflect/array.rs 62.50% 9 Missing ⚠️
ristretto_cli/src/main.rs 0.00% 1 Missing ⚠️
...vm/src/intrinsic_methods/java/io/filedescriptor.rs 50.00% 1 Missing ⚠️
...intrinsic_methods/java/lang/invoke/methodhandle.rs 0.00% 1 Missing ⚠️
...ic_methods/java/lang/invoke/methodhandlenatives.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
- Coverage   94.16%   94.16%   -0.01%     
==========================================
  Files         580      580              
  Lines       58994    58953      -41     
==========================================
- Hits        55554    55515      -39     
+ Misses       3440     3438       -2     

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

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

@brianheineman brianheineman force-pushed the remove-value-to-methods branch from 8441636 to 4063196 Compare July 28, 2025 22:21
@brianheineman brianheineman merged commit 093a6dc into main Jul 28, 2025
69 of 71 checks passed
@brianheineman brianheineman deleted the remove-value-to-methods branch July 28, 2025 22:46
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