feat(cli): Allow specifying target filenames by globbing#12
Conversation
This commit introduces the ability to specify target test files in the CLI using glob patterns. It replaces the previous `--filter` argument with a more flexible mechanism that accepts multiple glob patterns as command-line arguments. - Adds the `Microsoft.Extensions.FileSystemGlobbing` package to the CLI project. - Implements a robust argument parser to distinguish between options and glob patterns. - Uses the `Matcher` class to find test files matching the provided glob patterns. - Defaults to the `**/*test*.cs` pattern for backward compatibility when no patterns are provided. - Updates the error message to display the correct glob pattern information.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Adds a source code comment to `cli/Program.cs` to explain the dependency on argument position for file glob pattern detection. This addresses the feedback from the pull request review.
Updates the source code comment in `cli/Program.cs` to more accurately describe how the argument parsing logic distinguishes between options, their values, and file glob patterns. This addresses the feedback from the pull request review.
Updates the source code comment in `cli/Program.cs` to acknowledge a limitation in the argument parsing logic. This addresses the feedback from the pull request review.
sator-imaging
force-pushed
the
feature/glob-file-selection-5678090869467451192
branch
from
ed07eaa to
340ba73
Compare
sator-imaging
deleted the
feature/glob-file-selection-5678090869467451192
branch
1 participant
This change allows users to specify target test files in the CLI using glob patterns, providing a more flexible and powerful way to select tests. It replaces the old
--filterargument with a more robust mechanism that accepts multiple glob patterns. The implementation uses theMicrosoft.Extensions.FileSystemGlobbinglibrary and includes a default pattern for backward compatibility.PR created automatically by Jules for task 5678090869467451192 started by @sator-imaging