magento/magento-coding-standard issues and pull requests

#496 - Escaping for XML is caught by DiscouragedFunctionSniff

Issue - State: open - Opened by fooman 3 months ago - 1 comment
Labels: enhancement

#495 - Small cleanup, removed duplicated keys from array

Pull Request - State: open - Opened by hostep 3 months ago

#494 - Add support for Rector 2.x

Pull Request - State: open - Opened by fooman 5 months ago - 1 comment

#493 - Please add support for Rector 2.x

Issue - State: open - Opened by fooman 5 months ago - 2 comments

#492 - Please make copyright header check better

Issue - State: open - Opened by hostep 5 months ago - 1 comment
Labels: enhancement

#491 - PHPCompatibility should check for PHP 8.3 and 8.4 compatibility

Issue - State: open - Opened by hostep 5 months ago - 2 comments
Labels: enhancement

#490 - Stricter Escape Checks

Issue - State: open - Opened by Morgy93 6 months ago - 2 comments
Labels: enhancement, Progress: ready for grooming

#489 - Remove PHP `assert()` function from list of forbidden functions

Issue - State: open - Opened by oleksii-lisovyi 7 months ago - 1 comment
Labels: enhancement, Progress: ready for grooming

#488 - Require a strict_types declaration

Pull Request - State: open - Opened by fredden 10 months ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#487 - Update supported PHP versions to match Magento

Pull Request - State: open - Opened by fredden 11 months ago - 1 comment
Labels: partners-contribution, Progress: approved, Partner: Youwe

#486 - Resolve issues with regex include and exclude patterns in Magento 2 ruleset.xml

Pull Request - State: open - Opened by gwharton 11 months ago - 1 comment
Labels: Progress: approved

#485 - Several include-path regex issues on both Windows and Linux

Issue - State: open - Opened by gwharton 11 months ago - 5 comments
Labels: bug, Progress: PR created

#484 - PHP 8.3 error mentioning php 8.1 as version

Issue - State: open - Opened by MaximGns 12 months ago - 5 comments
Labels: bug, Progress: PR created

#483 - Fix Class Property DocBlock sniff for union/intersection types

Pull Request - State: closed - Opened by CRC-Mismatch about 1 year ago - 4 comments

#482 - Ignore HTML comments

Issue - State: open - Opened by minsc78 about 1 year ago - 1 comment
Labels: bug, Progress: ready for grooming

#481 - Question on how are this coding standards used in the Quality Reports of Adobe Commerce Marketplace

Issue - State: open - Opened by madalinignisca about 1 year ago - 1 comment
Labels: enhancement, Progress: ready for grooming

#480 - AC-12039: Exclude xml files in "Generic.Files.LineLength" rule in Static Test ruleset

Pull Request - State: open - Opened by yaroslavGoncharuk about 1 year ago
Labels: Progress: ready for review

#479 - [Issue] Update ESLint configuration

Issue - State: open - Opened by m2-assistant[bot] about 1 year ago
Labels: Progress: PR created

#478 - Avoid false positive with 'use function ...' lines

Pull Request - State: open - Opened by fredden over 1 year ago
Labels: partners-contribution, Progress: ready for review, Partner: Youwe

#477 - Rule Magento2.Exceptions.TryProcessSystemResources should exclude warnings from "use function .." section

Issue - State: open - Opened by ed007m over 1 year ago - 3 comments
Labels: enhancement, Progress: PR created

#476 - Member variables and methods should not require DocBlock when properly typed

Pull Request - State: open - Opened by JacobBrownAustin over 1 year ago - 7 comments
Labels: Progress: review

#475 - Replace dependency on abandoned PHP_CodeSniffer tool

Pull Request - State: closed - Opened by JosephLeedy over 1 year ago - 3 comments

#474 - Replace SquizLabs PHPCodeSniffer with Fork

Issue - State: closed - Opened by JosephLeedy over 1 year ago - 7 comments
Labels: Progress: done

#473 - Allow installing latest coding standard on Magento 2.4.5

Pull Request - State: closed - Opened by ihor-sviziev over 1 year ago - 1 comment

#472 - Rector rule added to escape unsafe output in phtml files

Pull Request - State: open - Opened by Jakhotiya over 1 year ago - 3 comments
Labels: Progress: review

#471 - Allow multiline css declarations in less files for increased readability

Issue - State: open - Opened by hostep almost 2 years ago - 1 comment
Labels: bug, Progress: ready for grooming

#470 - Conflicts with PER-CS 2.0 in regards with formatting braces for empty classes

Issue - State: open - Opened by hostep almost 2 years ago - 2 comments
Labels: bug, Progress: ready for grooming

#469 - Allow more recent versions of rector

Pull Request - State: closed - Opened by fredden almost 2 years ago - 2 comments
Labels: partners-contribution, Progress: done, Partner: Youwe

#468 - Removing dependency on symfony/polyfill, is no longer required now th…

Pull Request - State: closed - Opened by hostep almost 2 years ago - 5 comments
Labels: Progress: done

#467 - Individual polyfill packages should be used instead of symfony/polyfill

Issue - State: closed - Opened by stof almost 2 years ago - 5 comments
Labels: bug, Progress: done

#466 - Allow graphql pretty formatted files

Pull Request - State: open - Opened by p-makowski almost 2 years ago - 12 comments
Labels: partners-contribution, Progress: approved, Partner: Visma Digital Commerce

#465 - Warnings on well formatted GraphQL files

Issue - State: open - Opened by p-makowski almost 2 years ago - 2 comments
Labels: bug, Progress: PR created

#464 - Pinned composer dependency to rector 0.17.12

Issue - State: closed - Opened by cmuench almost 2 years ago - 1 comment
Labels: bug, Progress: done

#463 - Acmark

Pull Request - State: closed - Opened by kovarjan almost 2 years ago

#462 - Enforce import of global functions or using FQN calls

Issue - State: open - Opened by ilnytskyi almost 2 years ago - 1 comment
Labels: proposal, Progress: ready for grooming

#461 - Omit unnecessary files from distribution archive

Pull Request - State: open - Opened by fredden almost 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#460 - DiscouragedFunctionUnitTest.inc Security Flag

Issue - State: open - Opened by drinkingsouls almost 2 years ago - 5 comments
Labels: Progress: PR created

#459 - Update README.md

Pull Request - State: closed - Opened by snoop0x7b about 2 years ago - 4 comments
Labels: Progress: done

#458 - Add sniff for deprecated use of $block->escape... methods

Pull Request - State: closed - Opened by fredden about 2 years ago - 2 comments
Labels: partners-contribution, Progress: done, Partner: Youwe

#457 - Respect `--basepath` configuration option in output

Pull Request - State: open - Opened by fredden about 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#456 - Less validate check seem incorrectly as expected

Issue - State: open - Opened by mrtuvn about 2 years ago - 1 comment
Labels: bug, Progress: ready for grooming

#455 - Make fixer available via FileProxy

Pull Request - State: closed - Opened by fredden about 2 years ago
Labels: partners-contribution, Partner: Youwe

#454 - Tests are failing on develop branch

Issue - State: closed - Opened by ihor-sviziev about 2 years ago - 2 comments
Labels: bug, Progress: done

#453 - Update supported PHP versions

Pull Request - State: closed - Opened by fredden about 2 years ago - 28 comments
Labels: partners-contribution, Progress: done, Partner: Youwe

#452 - Remove local copies of PHPCompatibility sniffs

Pull Request - State: closed - Opened by fredden about 2 years ago - 1 comment
Labels: partners-contribution, Progress: done, Partner: Youwe

#451 - Please remove all stolen PHPCompatibilily sniffs

Issue - State: closed - Opened by jrfnl about 2 years ago - 16 comments
Labels: Progress: done

#450 - Auto-fix `Magento2.PHP.LiteralNamespaces`

Pull Request - State: open - Opened by fredden about 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#449 - Auto-fix `Magento2.Whitespace.MultipleEmptyLines`

Pull Request - State: open - Opened by fredden about 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#448 - Auto-fix `Magento2.Legacy.PhtmlTemplate.TextJavascriptTypeFound`

Pull Request - State: open - Opened by fredden about 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#447 - Allow readonly between phpdoc and property name

Pull Request - State: closed - Opened by sandipklevu about 2 years ago - 3 comments
Labels: Progress: done

#446 - Throws Missing PHP DocBlock for class property warning if readonly used in class property | PHP 8.1

Issue - State: closed - Opened by sandipklevu about 2 years ago - 1 comment
Labels: bug, Progress: done

#445 - Add suggested replacement for \posix_isatty()

Pull Request - State: closed - Opened by fredden about 2 years ago - 2 comments
Labels: partners-contribution, Progress: done, Partner: Youwe

#444 - Magento2\Sniffs\Legacy\RestrictedCodeSniff doesn't catch namespaced classes

Issue - State: open - Opened by DmitryFurs about 2 years ago - 1 comment
Labels: bug, Progress: ready for grooming

#443 - Also lint HTML when not at start of file

Pull Request - State: open - Opened by fredden over 2 years ago
Labels: partners-contribution, Progress: ready for review, Partner: Youwe

#442 - Make Unescaped Output Fail With Error Severity 10

Pull Request - State: closed - Opened by pykettk over 2 years ago - 4 comments
Labels: partners-contribution, Progress: done, Partner: Youwe

#441 - Allow use of LESS variables in "class name"

Pull Request - State: open - Opened by fredden over 2 years ago - 3 comments
Labels: partners-contribution, Progress: approved, Partner: Youwe

#440 - Allow no comment for a constructor where needed

Pull Request - State: open - Opened by fredden over 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#439 - Allow more version numbers in `@deprecated` tag

Pull Request - State: open - Opened by fredden over 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Fisheye

#438 - Auto-fix `Magento2.PHP.ShortEchoSyntax.ShortEchoTag`

Pull Request - State: closed - Opened by fredden over 2 years ago - 2 comments
Labels: partners-contribution, Progress: done, Partner: Fisheye

#437 - Ensure only the relevant docblock is read

Pull Request - State: closed - Opened by fredden over 2 years ago - 4 comments
Labels: partners-contribution, Progress: done, Partner: Fisheye

#436 - Simplify installation instructions

Pull Request - State: open - Opened by fredden over 2 years ago - 2 comments
Labels: partners-contribution, Progress: approved, Partner: Fisheye

#435 - Improve developer experience with closing tags

Pull Request - State: open - Opened by fredden over 2 years ago
Labels: partners-contribution, Progress: approved, Partner: Fisheye

#434 - Less files: valid Less contruction parsing issue

Issue - State: open - Opened by JuiceBlynskyi over 2 years ago - 1 comment
Labels: bug, Progress: PR created

#433 - Automatically remove useless comments

Pull Request - State: closed - Opened by fredden over 2 years ago - 6 comments
Labels: partners-contribution, Progress: done, Partner: Fisheye

#432 - Allow class names with 'admin__' prefix

Pull Request - State: closed - Opened by fredden over 2 years ago - 2 comments
Labels: partners-contribution, Progress: done, Partner: Fisheye

#431 - Restrict Union and Intersection types in public methods

Issue - State: closed - Opened by barbazul over 2 years ago - 9 comments
Labels: proposal, Progress: done

#430 - autotests fail on mainbranch

Pull Request - State: closed - Opened by victor-v-rad over 2 years ago - 1 comment

#429 - AC-7098: Performance issue in TableNameSniff with big array

Pull Request - State: closed - Opened by victor-v-rad over 2 years ago - 2 comments

#428 - RestrictedCodeSniff suggests Laminas\Filter\FilterInterface, which is not required by Magento 2.4.5-p1 anymore

Issue - State: open - Opened by norgeindian over 2 years ago - 7 comments
Labels: bug, Progress: ready for grooming

#427 - Exception class Magento\Framework\Validator\ValidateException does not exist

Issue - State: open - Opened by okolya almost 3 years ago - 2 comments
Labels: Progress: ready for grooming

#426 - Fix PHP error with short class names

Pull Request - State: closed - Opened by fredden almost 3 years ago - 6 comments
Labels: partners-contribution, Progress: done, Partner: Fisheye

#425 - strpos(): Argument #3 ($offset) must be contained in argument #1 ($haystack) in Magento2/Sniffs/Less/ClassNamingSniff.php:69

Issue - State: closed - Opened by lykhouzov almost 3 years ago - 1 comment
Labels: bug, Progress: done

GitHub / magento/magento-coding-standard issues and pull requests