magento/magento-coding-standard issues and pull requests

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

Pull Request - State: open - Opened by yaroslavGoncharuk 4 months ago
Labels: Progress: ready for review

#479 - [Issue] Update ESLint configuration

Issue - State: open - Opened by m2-assistant[bot] 5 months ago
Labels: Progress: PR created

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

Pull Request - State: open - Opened by fredden 6 months 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 7 months 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 10 months ago - 6 comments
Labels: Progress: review

#475 - Replace dependency on abandoned PHP_CodeSniffer tool

Pull Request - State: closed - Opened by JosephLeedy 10 months ago - 3 comments

#474 - Replace SquizLabs PHPCodeSniffer with Fork

Issue - State: closed - Opened by JosephLeedy 10 months ago - 7 comments
Labels: Progress: done

#473 - Allow installing latest coding standard on Magento 2.4.5

Pull Request - State: open - Opened by ihor-sviziev 10 months ago
Labels: Progress: ready for review

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

Pull Request - State: open - Opened by Jakhotiya 11 months ago - 3 comments
Labels: Progress: review

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

Issue - State: open - Opened by hostep about 1 year 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 about 1 year ago - 2 comments
Labels: bug, Progress: ready for grooming

#469 - Allow more recent versions of rector

Pull Request - State: closed - Opened by fredden about 1 year 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 about 1 year ago - 5 comments
Labels: Progress: done

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

Issue - State: closed - Opened by stof about 1 year ago - 5 comments
Labels: bug, Progress: done

#466 - Allow graphql pretty formatted files

Pull Request - State: open - Opened by p-makowski about 1 year 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 about 1 year ago - 2 comments
Labels: bug, Progress: PR created

#464 - Pinned composer dependency to rector 0.17.12

Issue - State: closed - Opened by cmuench about 1 year ago - 1 comment
Labels: bug, Progress: done

#463 - Acmark

Pull Request - State: closed - Opened by kovarjan about 1 year ago

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

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

#461 - Omit unnecessary files from distribution archive

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

#460 - DiscouragedFunctionUnitTest.inc Security Flag

Issue - State: open - Opened by drinkingsouls about 1 year ago - 5 comments
Labels: Progress: PR created

#459 - Update README.md

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

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

Pull Request - State: closed - Opened by fredden about 1 year 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 1 year ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

#456 - Less validate check seem incorrectly as expected

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

#455 - Make fixer available via FileProxy

Pull Request - State: closed - Opened by fredden over 1 year ago
Labels: partners-contribution, Partner: Youwe

#454 - Tests are failing on develop branch

Issue - State: closed - Opened by ihor-sviziev over 1 year ago - 2 comments
Labels: bug, Progress: done

#453 - Update supported PHP versions

Pull Request - State: closed - Opened by fredden over 1 year ago - 28 comments
Labels: partners-contribution, Progress: done, Partner: Youwe

#452 - Remove local copies of PHPCompatibility sniffs

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

#451 - Please remove all stolen PHPCompatibilily sniffs

Issue - State: closed - Opened by jrfnl over 1 year ago - 16 comments
Labels: Progress: done

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

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

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

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

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

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

#447 - Allow readonly between phpdoc and property name

Pull Request - State: closed - Opened by sandipklevu over 1 year 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 over 1 year ago - 1 comment
Labels: bug, Progress: done

#445 - Add suggested replacement for \posix_isatty()

Pull Request - State: closed - Opened by fredden over 1 year 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 over 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago
Labels: partners-contribution, Progress: approved, Partner: Youwe

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

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

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

Pull Request - State: closed - Opened by fredden over 1 year 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 1 year ago - 4 comments
Labels: partners-contribution, Progress: done, Partner: Fisheye

#436 - Simplify installation instructions

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

#435 - Improve developer experience with closing tags

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

#434 - Less files: valid Less contruction parsing issue

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

#433 - Automatically remove useless comments

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

#432 - Allow class names with 'admin__' prefix

Pull Request - State: closed - Opened by fredden over 1 year 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 1 year ago - 9 comments
Labels: proposal, Progress: done

#430 - autotests fail on mainbranch

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

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

Pull Request - State: closed - Opened by victor-v-rad almost 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 almost 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 2 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 2 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 2 years ago - 1 comment
Labels: bug, Progress: done

#424 - Forbid Framework\Module\Status::setIsEnabled

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

#423 - Too strict or early rule for Zend_Validate

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

#422 - Bump to version 26

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

#421 - Release v26

Pull Request - State: closed - Opened by bl4de almost 2 years ago
Labels: Progress: done

#420 - Add eliminated classes to Magento coding standards in Legacy\RestrictedCodeSniff

Pull Request - State: closed - Opened by karyna-tsymbal-atwix about 2 years ago - 4 comments
Labels: partners-contribution, Partner: Atwix, Progress: done

#419 - Allow to skip see tag in annotation with specific deprecated comment

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

#418 - PHP imports

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

#417 - Include all Magento2 sniffs in Magento2Framework

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

#416 - Upgrade PSR2 -> PSR12 for ControlStructureSpacing

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

#415 - Allow auto-fixing `@inheritDoc` tag when alone

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

#414 - Cannot switch to latest coding standard

Issue - State: open - Opened by unicoder88 about 2 years ago - 5 comments
Labels: bug, Progress: ready for grooming

#413 - Only auto-fix `$this` to `$block` when safe to do

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

#412 - PHP8.1 / PHPCS Empty FUNCTION statement detected with constructor property promotion

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

#411 - False Positive: Magento2.Legacy.InstallUpgrade.ObsoleteInstallDataScript

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

#410 - Fix Rector tests

Pull Request - State: closed - Opened by bl4de over 2 years ago - 2 comments
Labels: Progress: done

#409 - Less files : Class name separator issue

Issue - State: closed - Opened by Nuranto over 2 years ago - 10 comments
Labels: bug, Progress: done

#408 - Prevent ESlint from using .eslintrc config file(s) outside of eslint/rules

Pull Request - State: closed - Opened by bl4de over 2 years ago

#407 - Replacing `$this` with `$block` in templates is not universally safe / correct

Issue - State: open - Opened by fredden over 2 years ago - 5 comments
Labels: bug, Progress: PR created

#406 - Class properties that have types should not require a DocBlock

Pull Request - State: open - Opened by aligent-lturner over 2 years ago - 7 comments
Labels: partners-contribution, Progress: approved, Partner: Aligent Consulting

#405 - AC-3520: Avoid processing less and CSS pseudo-classes

Pull Request - State: closed - Opened by svera over 2 years ago - 2 comments
Labels: Progress: done

#404 - Do not require DocBlock for typed class attributes

Issue - State: open - Opened by aligent-lturner over 2 years ago - 1 comment
Labels: enhancement, Progress: PR created

#403 - [Issue] Allow Use of Template Literals

Issue - State: closed - Opened by m2-assistant[bot] over 2 years ago
Labels: Progress: done

#402 - "Avoid using self-closing tag" rule is false positive with inline svg

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

#401 - Is there any reason why function `parse_url` is Discouraged?

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

#400 - Check @deprecated format only if file has comment

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

#399 - Calling hasDeprecatedWellFormatted with a false $commentStartPtr leads to error

Issue - State: closed - Opened by mattijv over 2 years ago - 2 comments
Labels: bug, Progress: done

#398 - Exclude xml files for LineLength rule ?

Issue - State: open - Opened by Nuranto over 2 years ago - 3 comments
Labels: enhancement, Progress: ready for grooming

#397 - AC-3100: Add the @deprecated and @see tag check to ClassPropertyPHPDocFormattingSniff

Pull Request - State: closed - Opened by loginesta over 2 years ago - 2 comments
Labels: Progress: done

#396 - AC-3099: Add a sniff for functions to display a warning when there is no @see tag

Pull Request - State: closed - Opened by loginesta over 2 years ago - 2 comments
Labels: Progress: done

#395 - Expected 1 space after colon with &:not(:first-child)

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

#394 - dummy test

Pull Request - State: closed - Opened by loginesta over 2 years ago - 1 comment

#393 - AC-3098: Update hasDeprecatedWellFormatted

Pull Request - State: closed - Opened by loginesta over 2 years ago - 2 comments
Labels: Progress: done

#392 - Reactor rule php8 ReplaceNewDateTimeNull has wrong implementation

Issue - State: open - Opened by 0m3r over 2 years ago - 1 comment
Labels: enhancement, Progress: ready for grooming

#391 - phpcs: PHP Fatal error: Uncaught Error: Undefined constant "Magento2\Sniffs\Annotation\T_ATTRIBUTE_END"

Issue - State: closed - Opened by tuyennn over 2 years ago - 2 comments
Labels: bug, Progress: done

#390 - PHPStorm unable to run phpcs inspection

Issue - State: open - Opened by Vinai over 2 years ago - 12 comments
Labels: bug, Progress: PR created

#389 - Release v22

Pull Request - State: closed - Opened by loginesta over 2 years ago

#388 - Magento sniff codes do not match expected format (too many `.`)

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

#387 - Allow empty constructor function blocks when using property promotion

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

#386 - AC-1879: Magento2/Sniffs/Legacy/ObsoleteResponseSniff.php issue

Pull Request - State: closed - Opened by bl4de over 2 years ago - 2 comments
Labels: Progress: done

#385 - Constructor property promotion and EmptyBlock rule

Issue - State: closed - Opened by mfickers over 2 years ago - 2 comments
Labels: enhancement, Progress: done

#384 - AC-2724: Update rector dependency to the latest version

Pull Request - State: closed - Opened by eliseacornejo over 2 years ago - 2 comments
Labels: Progress: done

#383 - Fix warnings detected by phpcs & phpunit

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

#382 - Add `@author` to forbidden tag list

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

#381 - Release v21

Pull Request - State: closed - Opened by loginesta over 2 years ago

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