Skip to content

adding tools.rs, add extract_i64 #635

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 25, 2023
Merged

adding tools.rs, add extract_i64 #635

merged 4 commits into from
May 25, 2023

Conversation

samuelcolvin
Copy link
Member

@samuelcolvin samuelcolvin commented May 25, 2023

Selected Reviewer: @adriangb

@codspeed-hq
Copy link

codspeed-hq bot commented May 25, 2023

CodSpeed Performance Report

Merging #635 tools (3193163) will not alter performances.

Summary

🔥 0 improvements
❌ 0 regressions
✅ 121 untouched benchmarks

🆕 0 new benchmarks
⁉️ 0 dropped benchmarks

@codecov-commenter
Copy link

codecov-commenter commented May 25, 2023

Codecov Report

Merging #635 (fb694d0) into main (26fa27d) will increase coverage by 0.00%.
The diff coverage is 82.30%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #635   +/-   ##
=======================================
  Coverage   94.04%   94.05%           
=======================================
  Files          98       99    +1     
  Lines       13456    13466   +10     
  Branches       25       25           
=======================================
+ Hits        12655    12665   +10     
  Misses        795      795           
  Partials        6        6           
Impacted Files Coverage Δ
src/argument_markers.rs 87.87% <ø> (ø)
src/lib.rs 100.00% <ø> (ø)
src/serializers/config.rs 93.75% <0.00%> (ø)
src/serializers/filter.rs 96.07% <ø> (ø)
src/serializers/type_serializers/definitions.rs 93.22% <ø> (ø)
src/serializers/type_serializers/dict.rs 95.55% <ø> (ø)
src/serializers/type_serializers/format.rs 91.11% <0.00%> (ø)
src/serializers/type_serializers/generator.rs 93.47% <ø> (ø)
src/serializers/type_serializers/json.rs 96.49% <ø> (ø)
src/serializers/type_serializers/json_or_python.rs 85.71% <ø> (ø)
... and 59 more

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 26fa27d...fb694d0. Read the comment docs.

@samuelcolvin samuelcolvin changed the title adding tools.rs adding tools.rs, add extract_i64 May 25, 2023
@samuelcolvin
Copy link
Member Author

please review.

@samuelcolvin samuelcolvin merged commit 6220455 into main May 25, 2023
@samuelcolvin samuelcolvin deleted the tools branch May 25, 2023 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants