Skip to content

Conversation

@konard
Copy link
Member

@konard konard commented Sep 13, 2025

Summary

Renamed the DeleteAll method in FileHelpers to DeleteFiles as requested in issue #53.

  • Renamed DeleteAll method to DeleteFiles in both C# (csharp/Platform.IO/FileHelpers.cs) and C++ (cpp/Platform.IO/FileHelpers.h) implementations
  • Updated all three overloads of the method:
    • DeleteFiles(string directory) - defaults to "*" pattern
    • DeleteFiles(string directory, string searchPattern) - defaults to TopDirectoryOnly
    • DeleteFiles(string directory, string searchPattern, SearchOption searchOption) - full control
  • Added comprehensive test coverage for the renamed method in FileHelpersTests.cs
  • Verified functionality with both pattern-specific deletion and wildcard behavior

Test plan

  • All existing tests pass
  • New test DeleteFilesTest validates:
    • Pattern-specific file deletion (*.txt files only)
    • Wildcard deletion (all remaining files)
    • Proper cleanup and directory handling
  • Build completes successfully for entire solution

The method name DeleteFiles was chosen over DeleteMany as it more clearly describes the method's purpose - specifically deleting files from directories based on search patterns.

🤖 Generated with Claude Code


Resolves #53

Adding CLAUDE.md with task information for AI processing.
This file will be removed when the task is complete.

Issue: #53
@konard konard self-assigned this Sep 13, 2025
- Renamed DeleteAll method to DeleteFiles in both C# and C++ implementations
- Updated all three overloads of the method in FileHelpers
- Added comprehensive tests for the renamed method functionality
- Verified both specific pattern matching and default wildcard behavior

This change improves method naming clarity as the method specifically
deletes files from directories rather than all types of entities.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@konard konard changed the title [WIP] Rename DeleteAll method Rename DeleteAll method to DeleteFiles Sep 13, 2025
@konard konard marked this pull request as ready for review September 13, 2025 15:04
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.

Rename DeleteAll method

2 participants