Skip to content

Conversation

samuelcolvin
Copy link
Member

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Apr 1, 2023

Codecov Report

Merging #510 (dfeecdb) into main (e65b46f) will increase coverage by 0.06%.
The diff coverage is 87.93%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #510      +/-   ##
==========================================
+ Coverage   94.65%   94.72%   +0.06%     
==========================================
  Files          93       93              
  Lines       12171    12216      +45     
  Branches       25       25              
==========================================
+ Hits        11521    11572      +51     
+ Misses        643      637       -6     
  Partials        7        7              
Impacted Files Coverage Δ
src/errors/location.rs 97.24% <84.21%> (-1.69%) ⬇️
src/errors/validation_exception.rs 93.46% <85.18%> (+1.61%) ⬆️
src/errors/types.rs 84.33% <100.00%> (+2.01%) ⬆️
src/errors/value_exception.rs 94.84% <100.00%> (ø)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e65b46f...dfeecdb. Read the comment docs.

@codspeed-hq
Copy link

codspeed-hq bot commented Apr 1, 2023

CodSpeed Performance Report

Merging #510 raising-ValidationError (dfeecdb) will not alter performances.

Summary

🔥 0 improvements
❌ 0 regressions
✅ 95 untouched benchmarks

🆕 0 new benchmarks
⁉️ 0 dropped benchmarks

@samuelcolvin samuelcolvin merged commit fd987e8 into main Apr 1, 2023
@samuelcolvin samuelcolvin deleted the raising-ValidationError branch April 1, 2023 17:20
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