Skip to content
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

#720 - New Results API #723

Merged
merged 1 commit into from Sep 13, 2023
Merged

#720 - New Results API #723

merged 1 commit into from Sep 13, 2023

Conversation

sys27
Copy link
Owner

@sys27 sys27 commented Sep 12, 2023

Closes #720.

@sys27 sys27 added this to the xFunc 4.4.0 milestone Sep 12, 2023
@sys27 sys27 self-assigned this Sep 12, 2023
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch coverage: 99.52% and no project coverage change.

Comparison is base (fd94f7d) 99.65% compared to head (86d35c4) 99.65%.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #723   +/-   ##
=======================================
  Coverage   99.65%   99.65%           
=======================================
  Files         219      221    +2     
  Lines        8620     8707   +87     
=======================================
+ Hits         8590     8677   +87     
  Misses         30       30           
Files Changed Coverage Δ
xFunc.Maths/Results/Result.cs 98.24% <98.24%> (ø)
xFunc.Maths/Expressions/EmptyValue.cs 100.00% <100.00%> (ø)
xFunc.Maths/Expressions/Programming/For.cs 100.00% <100.00%> (ø)
xFunc.Maths/Expressions/Programming/While.cs 96.66% <100.00%> (ø)
xFunc.Maths/Processor.cs 100.00% <100.00%> (+1.02%) ⬆️
xFunc.Maths/Results/Result.AngleResult.cs 100.00% <100.00%> (ø)
xFunc.Maths/Results/Result.AreaResult.cs 100.00% <100.00%> (ø)
xFunc.Maths/Results/Result.BooleanResult.cs 100.00% <100.00%> (ø)
xFunc.Maths/Results/Result.ComplexNumberResult.cs 100.00% <100.00%> (ø)
xFunc.Maths/Results/Result.LambdaResult.cs 100.00% <100.00%> (ø)
... and 11 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sys27 sys27 marked this pull request as ready for review September 12, 2023 23:26
@sys27 sys27 force-pushed the feature/new-results-api branch 2 times, most recently from 92ad27f to 3fd0b07 Compare September 13, 2023 10:23
@sys27 sys27 changed the title #720 - New Results API. #720 - New Results API Sep 13, 2023
@sys27 sys27 force-pushed the feature/new-results-api branch 3 times, most recently from 3a490d6 to 57f2d0c Compare September 13, 2023 15:25
@sys27 sys27 merged commit d957811 into dev Sep 13, 2023
4 of 5 checks passed
@sys27 sys27 deleted the feature/new-results-api branch September 13, 2023 15:36
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.

New Results API
1 participant