Skip to content

A few clippy fixes#5424

Open
MatrixFrog wants to merge 1 commit into
boa-dev:mainfrom
MatrixFrog:clippy
Open

A few clippy fixes#5424
MatrixFrog wants to merge 1 commit into
boa-dev:mainfrom
MatrixFrog:clippy

Conversation

@MatrixFrog

Copy link
Copy Markdown

These were flagged when trying to push to github

@MatrixFrog MatrixFrog requested review from a team, jedel1043 and nekevss as code owners July 4, 2026 16:31
@github-actions github-actions Bot added the Waiting On Review Waiting on reviews from the maintainers label Jul 4, 2026
@github-actions github-actions Bot added this to the v1.0.0 milestone Jul 4, 2026
@github-actions github-actions Bot added C-Builtins PRs and Issues related to builtins/intrinsics C-AST Issue surrounding the abstract syntax tree C-GC Issue related to garbage collection C-Intl Changes related to the `Intl` implementation labels Jul 4, 2026
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

Test262 conformance changes

Test result main count PR count difference
Total 53,125 53,125 0
Passed 51,072 51,072 0
Ignored 1,482 1,482 0
Failed 571 571 0
Panics 0 0 0
Conformance 96.14% 96.14% 0.00%

Tested main commit: 8a1e8fe07f626f7a067afc2c9885d5d87de4bb5d
Tested PR commit: b131244064056ff6622326ca1067ece6639e0d08
Compare commits: 8a1e8fe...b131244

@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.70%. Comparing base (6ddc2b4) to head (b131244).
⚠️ Report is 985 commits behind head on main.

Files with missing lines Patch % Lines
core/ast/src/expression/literal/object.rs 83.33% 2 Missing ⚠️
core/string/src/builder.rs 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #5424       +/-   ##
===========================================
+ Coverage   47.24%   62.70%   +15.45%     
===========================================
  Files         476      535       +59     
  Lines       46892    59083    +12191     
===========================================
+ Hits        22154    37047    +14893     
+ Misses      24738    22036     -2702     

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-AST Issue surrounding the abstract syntax tree C-Builtins PRs and Issues related to builtins/intrinsics C-GC Issue related to garbage collection C-Intl Changes related to the `Intl` implementation Waiting On Review Waiting on reviews from the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant