Skip to content

Refactor rules: configurable suffixes, message registry, InClassNode - #6

Merged
Simtel merged 1 commit into
masterfrom
3.1
Aug 26, 2026
Merged

Simtel merged 1 commit into
masterfrom
3.1

Conversation

@Simtel

@Simtel Simtel commented Aug 26, 2026

Copy link
Copy Markdown
Owner
  • Make class name suffixes configurable via constructor args (Command, CommandHandler, EventListener, AsEventListener) with defaults
  • Add RuleMessages registry for error templates and identifiers
  • Switch class-level rules to InClassNode and use ClassReflection (getDisplayName, hasNativeMethod, getAttributes) instead of raw AST names
  • Convert command rule test to a #[DataProvider] dataset
  • Document suffix customization in README

- Make class name suffixes configurable via constructor args (Command,
  CommandHandler, EventListener, AsEventListener) with defaults
- Add RuleMessages registry for error templates and identifiers
- Switch class-level rules to InClassNode and use ClassReflection
  (getDisplayName, hasNativeMethod, getAttributes) instead of raw AST names
- Convert command rule test to a #[DataProvider] dataset
- Document suffix customization in README
@Simtel
Simtel merged commit dc7aff8 into master Aug 26, 2026
1 check passed
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.

1 participant