Skip to content

642 add sizeof builtin function#672

Merged
mhasel merged 6 commits intomasterfrom
642-add-sizeof-builtin-function
Dec 5, 2022
Merged

642 add sizeof builtin function#672
mhasel merged 6 commits intomasterfrom
642-add-sizeof-builtin-function

Conversation

@mhasel
Copy link
Member

@mhasel mhasel commented Dec 2, 2022

Adds builtin sizeof function and fixes various clippy lints (chrono deprecated function calls, ...) and some typos

@mhasel mhasel linked an issue Dec 2, 2022 that may be closed by this pull request
Copy link
Collaborator

@ghaith ghaith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, especially with fixing all my spelling mistakes 👍

@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2022

Codecov Report

Base: 94.00% // Head: 93.99% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (d15ccc9) compared to base (31c536e).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@ Coverage Diff @@ ## master #672 +/- ## ========================================== - Coverage 94.00% 93.99% -0.02%  ========================================== Files 46 46 Lines 18223 18241 +18 ========================================== + Hits 17130 17145 +15  - Misses 1093 1096 +3 
Impacted Files Coverage Δ
src/builtins.rs 86.84% <75.00%> (-1.22%) ⬇️
src/validation.rs 98.05% <0.00%> (-0.28%) ⬇️
src/parser/expressions_parser.rs 93.96% <0.00%> (+0.12%) ⬆️
src/resolver/generics.rs 96.82% <0.00%> (+0.28%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mhasel mhasel merged commit f7e8159 into master Dec 5, 2022
@mhasel mhasel deleted the 642-add-sizeof-builtin-function branch December 5, 2022 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants