Skip to content

cleanup: remove mod.rs and other lints#347

Merged
slawlor merged 1 commit intomainfrom
fmt
Apr 24, 2025
Merged

cleanup: remove mod.rs and other lints#347
slawlor merged 1 commit intomainfrom
fmt

Conversation

@slawlor
Copy link
Copy Markdown
Owner

@slawlor slawlor commented Apr 24, 2025

Also this adjusts the unsafe code build failure such that additional,
non-allowlisted unsafe code will fail the build for safety.

Ractor should have a single unsafe entry in it, specifically for error
handling and auto-conversion.

Also this adjusts the unsafe code build failure such that additional,
non-allowlisted unsafe code will fail the build for safety.

Ractor should have a single unsafe entry in it, specifically for error
handling and auto-conversion.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2025

Codecov Report

Attention: Patch coverage is 25.00000% with 18 lines in your changes missing coverage. Please review.

Project coverage is 80.56%. Comparing base (346f05a) to head (6011044).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
ractor_cluster/src/net.rs 0.00% 12 Missing ⚠️
ractor_cluster/src/node.rs 0.00% 4 Missing ⚠️
ractor_cluster/src/net/listener.rs 0.00% 1 Missing ⚠️
ractor_cluster/src/node/node_session.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #347      +/-   ##
==========================================
- Coverage   80.62%   80.56%   -0.06%     
==========================================
  Files          66       66              
  Lines       13247    13256       +9     
==========================================
  Hits        10680    10680              
- Misses       2567     2576       +9     

☔ 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.

@slawlor slawlor marked this pull request as ready for review April 24, 2025 18:57
@slawlor slawlor merged commit b1329e2 into main Apr 24, 2025
19 of 21 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.

1 participant