Skip to content

Task/48 PHP Mess detector issues#58

Merged
omerida merged 3 commits intomainfrom
task/48-phpmd-report
Dec 10, 2025
Merged

Task/48 PHP Mess detector issues#58
omerida merged 3 commits intomainfrom
task/48-phpmd-report

Conversation

@omerida
Copy link
Owner

@omerida omerida commented Dec 10, 2025

Linked Issue(s)

Fixes #48

Summary of Changes

#48 Review PHPMD errors

  • Expand short variable names and fix came case method names.
  • Inject environment config to service container.
  • Document expected environment variables.
  • Split RoutesMapper into separate responsibilities to reduce coupling between objects.

Testing

Manual testing

Breaking Changes / Migration Notes

(If applicable, describe any backward-incompatible changes and provide instructions for upgrading.)

  • This change introduces breaking changes.
  • No breaking changes.

* Expand short variable names and fix came case method names.
* Document expected environment variables.
@omerida omerida merged commit 62afb0f into main Dec 10, 2025
1 check passed
@omerida omerida deleted the task/48-phpmd-report branch December 11, 2025 05:20
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.

[MAINT] Review PHP Mess detector report

1 participant