Skip to content

Comments

Make namespace filters configurable in ReportCommand#11

Merged
makomweb merged 5 commits intonextfrom
copilot/sub-pr-10
Jan 25, 2026
Merged

Make namespace filters configurable in ReportCommand#11
makomweb merged 5 commits intonextfrom
copilot/sub-pr-10

Conversation

Copy link
Contributor

Copilot AI commented Jan 25, 2026

The ReportCommand contained hardcoded namespace filters specific to a single project (App\Kernel, App\CLI, PhpParser, Monolog, etc.), making it non-reusable without modification.

Changes

  • Added --exclude-namespace option (repeatable) to configure namespace exclusions
  • Replaced hardcoded project-specific filters with minimal general-purpose defaults (Doctrine, Symfony, Psr)
  • User-provided exclusions replace defaults entirely, allowing full control
  • Updated README with usage examples

Usage

# Use defaults (Doctrine, Symfony, Psr)
vendor/bin/console tactix:report src/

# Custom exclusions (replaces defaults)
vendor/bin/console tactix:report src/ \
  --exclude-namespace="App\\CLI\\" \
  --exclude-namespace="App\\Infrastructure\\"

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits January 25, 2026 14:44
Co-authored-by: makomweb <1567373+makomweb@users.noreply.github.com>
Co-authored-by: makomweb <1567373+makomweb@users.noreply.github.com>
Co-authored-by: makomweb <1567373+makomweb@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on CLI command to generate a report Make namespace filters configurable in ReportCommand Jan 25, 2026
Copilot AI requested a review from makomweb January 25, 2026 14:47
Copy link
Owner

@makomweb makomweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@makomweb makomweb marked this pull request as ready for review January 25, 2026 14:53
@makomweb makomweb merged commit 7bcab76 into next Jan 25, 2026
@makomweb makomweb deleted the copilot/sub-pr-10 branch January 25, 2026 14:53
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.

2 participants