Added exceptions for processing AI agent folders#306
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 554a125823
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ->withImportNames( | ||
| removeUnusedImports: true, | ||
| ) | ||
| ->withImportNames() |
There was a problem hiding this comment.
Preserve unused-import removal for supported Rector versions
This drops removeUnusedImports: true even though this package reaches Rector via driftingly/rector-laravel, whose supported range still allows rector/rector:^2.2.7; Rector only made unused-import removal the default in 2.5 (the release notes say “Removing unused imports by default”). In projects with a lockfile or constraint keeping Rector 2.2–2.4, publishing this preset now stops removing unused use statements (the same regression is present in laravel.php), so keep the explicit flag or raise the minimum Rector version.
Useful? React with 👍 / 👎.
No description provided.