magento-coding-standard
magento-coding-standard copied to clipboard
Magento Coding Standard
### Preconditions 1. Have a class like this: ```php
Currently checks for GraphQL ValidTypeName assumes that type definitions are always one-liners. For a better readability it is much better to format GraphQL files so that they are multiline. ```...
When HTML is used as part of a `.phtml` file, it should be subject to the same rules as HTML used in a `.html` file. The additional test files here...
https://wiki.corp.adobe.com/pages/viewpage.action?spaceKey=MC&title=Proposal+to+change+magento-coding-standard+to+not+require+DocBlock+when+parameters+defined
The installation instructions for this coding standard are unnecessarily complex. Composer plugins are capable of performing the necessary registration automatically, without requiring the user to manually set up (or alter)...
While reviewing the Static Test failures in https://github.com/magento/magento2/pull/36976 (which are out of scope for the intended change), I noticed that the message required for a `@deprecated` tag does not support...
These files are only useful when working directly on this standard, and not when consuming the standard. When installing this standard via Composer, these files can safely be excluded. Closes...
This pull request allows installing the latest version of coding standards to Magento 2.4.5-p5. Currently it fails due to incompatibility with version of the `webonyx/graphql-php` package: ```bash $ composer why-not...
https://github.com/magento/magento-coding-standard/commit/9c2ac84d1fddfbc1f5c16762d42280d1b7ffaeb9 is the main purpose of this pull request. In order to get there cleanly, a number of refactoring commits were required. I recommend that this pull request is reviewed...
## Summary Often a codebase is handed over to you which does not follow all magento standards and phpcs starts flagging them. One such example is unsafe output in template...