Skip to content

Update documentation and enhance Opt/Result functionality#14

Merged
trippwill merged 3 commits intomainfrom
trippwill/opt-enhancements
Dec 19, 2024
Merged

Update documentation and enhance Opt/Result functionality#14
trippwill merged 3 commits intomainfrom
trippwill/opt-enhancements

Conversation

@trippwill
Copy link
Member

  • Improved README.md with new features and examples.
  • Refactored Opt.cs methods for easier creation and added filtering and transforming options.
  • Expanded Result.cs with new methods for safe execution and validation.
  • Added unit tests in Examples.cs for implicit conversions and behavior of Opt/Result types.
  • Enhanced OptTests.cs with tests for new Filter and AndThen methods.
  • Modified libanvl.Opt.csproj to allow unsafe code blocks.

- Improved README.md with new features and examples.
- Refactored Opt.cs methods for easier creation and added filtering and transforming options.
- Expanded Result.cs with new methods for safe execution and validation.
- Added unit tests in Examples.cs for implicit conversions and behavior of Opt/Result types.
- Enhanced OptTests.cs with tests for new Filter and AndThen methods.
- Modified libanvl.Opt.csproj to allow unsafe code blocks.
@codecov
Copy link

codecov bot commented Dec 19, 2024

Codecov Report

Attention: Patch coverage is 90.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/libanvl.Opt/Opt.cs 75.00% 1 Missing and 1 partial ⚠️
src/libanvl.Opt/Result.cs 93.33% 1 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
src/libanvl.Opt/OptExtensions.cs 100.00% <100.00%> (+100.00%) ⬆️
src/libanvl.Opt/Opt.cs 50.79% <75.00%> (+5.48%) ⬆️
src/libanvl.Opt/Result.cs 84.26% <93.33%> (+25.93%) ⬆️

@github-actions
Copy link

github-actions bot commented Dec 19, 2024

Test Results

  3 files  +  1    3 suites  +1   0s ⏱️ ±0s
 71 tests + 45   71 ✅ + 45  0 💤 ±0  0 ❌ ±0 
213 runs  +161  213 ✅ +161  0 💤 ±0  0 ❌ ±0 

Results for commit f2fc457. ± Comparison against base commit b9d0b17.

This pull request removes 1 and adds 46 tests. Note that renamed tests count towards both.
libanvl.opt.test.ResultTests ‑ Try_With_Arg_Should_Return_Err_If_Function_Throws
libanvl.opt.test.Examples ‑ Operator_False
libanvl.opt.test.Examples ‑ Operator_ImplicitConversion_FromOpt
libanvl.opt.test.Examples ‑ Operator_ImplicitConversion_ToOpt
libanvl.opt.test.Examples ‑ Operator_Or_Opt
libanvl.opt.test.Examples ‑ Operator_Or_Value
libanvl.opt.test.Examples ‑ Operator_True
libanvl.opt.test.Examples ‑ Result_Err
libanvl.opt.test.Examples ‑ Result_ImplicitConversion_Err
libanvl.opt.test.Examples ‑ Result_ImplicitConversion_Ok
libanvl.opt.test.Examples ‑ Result_Match_Err
…

♻️ This comment has been updated with latest results.

@trippwill trippwill merged commit a80d1cd into main Dec 19, 2024
14 checks passed
@trippwill trippwill deleted the trippwill/opt-enhancements branch December 19, 2024 01:55
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