Skip to content

Removing recursive option#54

Merged
byte2pixel merged 1 commit intomainfrom
issue/53/remove-recurse-option
Feb 23, 2026
Merged

Removing recursive option#54
byte2pixel merged 1 commit intomainfrom
issue/53/remove-recurse-option

Conversation

@byte2pixel
Copy link
Owner

closes #53

Remove the -r --recursive option from the CLI.

Clean up tests and xml docs. A new issue will be created for updating the README before the release of the next major version.

@github-actions
Copy link

Test Results

  4 files  ±0    4 suites  ±0   2m 42s ⏱️ +6s
133 tests  - 2  133 ✅  - 2  0 💤 ±0  0 ❌ ±0 
266 runs   - 4  266 ✅  - 4  0 💤 ±0  0 ❌ ±0 

Results for commit 630a991. ± Comparison against base commit 7684f99.

This pull request removes 3 and adds 1 tests. Note that renamed tests count towards both.
Serilog.Sinks.File.Encrypt.Cli.Tests.unit.DecryptCommandTests ‑ GivenInputPath_WhenValidDirectory_ThenValidationSucceeds
Serilog.Sinks.File.Encrypt.Cli.Tests.unit.InputResolverTests ‑ GivenDirectory_WithRecursiveFalse_DoesNotIncludeSubdirectoryFiles
Serilog.Sinks.File.Encrypt.Cli.Tests.unit.InputResolverTests ‑ GivenDirectory_WithRecursiveTrue_IncludesSubdirectoryFiles
Serilog.Sinks.File.Encrypt.Cli.Tests.unit.DecryptCommandTests ‑ GivenInputPath_WhenValidDirectoryNoPattern_ThenValidationFails

@byte2pixel byte2pixel merged commit 23ba4b5 into main Feb 23, 2026
4 checks passed
@byte2pixel byte2pixel deleted the issue/53/remove-recurse-option branch February 23, 2026 23:53
@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

Remove recurse option from CLI

1 participant