Repository navigation
Conversation
Co-authored-by: bfoujols <7000210+bfoujols@users.noreply.github.com>
…definitions Fix undefined-method calls flagged by DeepSource (PHP-E1002) in make:* commands
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| PHP | Oct 5, 2026 8:32a.m. | Review ↗ | |
| Shell | Oct 5, 2026 8:32a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
This pull request contains several small but important improvements to command classes and the request handling in the EduFramework. The changes mainly address method naming consistency, correct usage of class methods, and minor logic and initialization fixes.
Consistency and Correctness in Command Classes
AddArgumenttoaddArgumentin theconfigure()methods ofCreateApiCommand,CreateCliCommand, andCreateControllerCommandfor consistency with Symfony's API. [1] [2] [3]getNamesCollectionfrom a static call to an instance call inCreateControllerCommand'sexecute()method, ensuring proper method invocation.Logic and Initialization Improvements
StartCommandby calculating the number of banner lines and erasing only the necessary lines between animation frames, making the output cleaner.$handerproperty to an empty string in theRequestclass to prevent uninitialized variable issues.