Skip to content

feat: Replace Argument.ThrowIfNull with ArgumentNullException.ThrowIf…#401

Merged
samtrion merged 2 commits intomainfrom
feat/arguments
Mar 23, 2026
Merged

feat: Replace Argument.ThrowIfNull with ArgumentNullException.ThrowIf…#401
samtrion merged 2 commits intomainfrom
feat/arguments

Conversation

@samtrion
Copy link
Contributor

…Null for built-in null checking

  • Remove NetEvolve.Arguments using directives where no longer needed
  • Remove coverlet.collector from Directory.Packages.props and test project
  • Clean up unused using directives in several files

…Null for built-in null checking

- Remove NetEvolve.Arguments using directives where no longer needed
- Remove coverlet.collector from Directory.Packages.props and test project
- Clean up unused using directives in several files
@samtrion samtrion self-assigned this Mar 23, 2026
@samtrion samtrion requested a review from a team as a code owner March 23, 2026 20:57
@samtrion samtrion requested a review from benwirren March 23, 2026 20:57
@samtrion samtrion added state:ready for merge Indicates that a pull request has been reviewed and approved, and is ready to be merged into the mai type:feature Indicates a new feature or enhancement to be added. labels Mar 23, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 23, 2026

Warning

Rate limit exceeded

@samtrion has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 7 minutes and 20 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3b6a70bc-d58e-4ff8-a350-b021178d251b

📥 Commits

Reviewing files that changed from the base of the PR and between 0a58bb9 and 355d22d.

📒 Files selected for processing (13)
  • .csharpierignore
  • .editorconfig
  • .github/workflows/cicd.yml
  • Directory.Packages.props
  • src/NetEvolve.FluentValue/Constraints/ConstraintBase.cs
  • src/NetEvolve.FluentValue/Constraints/DefaultConstraint.cs
  • src/NetEvolve.FluentValue/Constraints/ParenthesisConstraint.cs
  • src/NetEvolve.FluentValue/Operators/AndOperator.cs
  • src/NetEvolve.FluentValue/Operators/NotOperator.cs
  • src/NetEvolve.FluentValue/Operators/OrOperator.cs
  • src/NetEvolve.FluentValue/Operators/XorOperator.cs
  • tests/NetEvolve.FluentValue.Tests.Unit/NetEvolve.FluentValue.Tests.Unit.csproj
  • tests/NetEvolve.FluentValue.Tests.Unit/TypeExtensionsTests.cs

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.54%. Comparing base (0a58bb9) to head (355d22d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #401   +/-   ##
=======================================
  Coverage   98.54%   98.54%           
=======================================
  Files          22       22           
  Lines         274      274           
  Branches       43       43           
=======================================
  Hits          270      270           
  Misses          1        1           
  Partials        3        3           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@samtrion samtrion merged commit 9daaf8a into main Mar 23, 2026
14 checks passed
@samtrion samtrion deleted the feat/arguments branch March 23, 2026 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:ready for merge Indicates that a pull request has been reviewed and approved, and is ready to be merged into the mai type:feature Indicates a new feature or enhancement to be added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant