GitHub / spryker/code-sniffer issues and pull requests
#378 - FRW-8801 Dropped PHP 8.0 support.
Pull Request -
State: open - Opened by olhalivitchuk 11 months ago
Labels: improvement
#377 - FRW-8526 Added CONTRIBUTING.md.
Pull Request -
State: closed - Opened by olhalivitchuk about 1 year ago
Labels: improvement, merge-only
#376 - FRW-8590 Fixed false negative error for specification annotation.
Pull Request -
State: closed - Opened by geega about 1 year ago
Labels: bug, ready-for-release
#375 - FRW-7647 Removed deprecated `getQueryContainer()` annotation validation.
Pull Request -
State: closed - Opened by olhalivitchuk over 1 year ago
Labels: bug, ready-for-release
#374 - FRW-7452 Enabled constructor property promotion.
Pull Request -
State: closed - Opened by vol4onok over 1 year ago
Labels: improvement
#373 - FRW-7253 Fixed false negative error for SuppressWarnings annotation.
Pull Request -
State: closed - Opened by geega over 1 year ago
Labels: bug
#372 - SDK-5711: Removed unused enableNeverTypeHint.
Pull Request -
State: closed - Opened by vol4onok over 1 year ago
- 1 comment
#371 - FRW-998 Dropped PHP 7.4 support.
Pull Request -
State: closed - Opened by asmarovydlo almost 2 years ago
Labels: improvement, ready-for-release
#370 - DisallowedArrayTypeHintSyntax is not compatible with slevomat/coding-standard >= 8.13.0
Issue -
State: open - Opened by alfredbez almost 2 years ago
#369 - Fix WrongNullable false positive
Pull Request -
State: open - Opened by hemberger over 2 years ago
- 2 comments
#368 - WrongNullable false positive
Issue -
State: open - Opened by hemberger over 2 years ago
- 1 comment
#367 - Enable Spryker.Commenting.DocBlockApiAnnotation for project level code
Issue -
State: open - Opened by alfredbez over 2 years ago
- 3 comments
#366 - Removed composer validate lowest check from CI
Pull Request -
State: closed - Opened by a-sabaa over 2 years ago
Labels: merge-only
#365 - Make namespace sniff more generic
Pull Request -
State: closed - Opened by dereuromark over 2 years ago
Labels: improvement
#364 - Fix up 7.4+
Pull Request -
State: closed - Opened by dereuromark over 2 years ago
Labels: merge-only
#363 - Fix false positive
Pull Request -
State: closed - Opened by dereuromark over 2 years ago
- 1 comment
Labels: bug, prio
#362 - InlineDocCommentDeclaration issue
Issue -
State: closed - Opened by dereuromark almost 3 years ago
- 1 comment
Labels: bug
#361 - PHP 82 check
Pull Request -
State: closed - Opened by dereuromark almost 3 years ago
Labels: merge-only
#360 - FRW-332 Change min PHP version to 7.4
Pull Request -
State: closed - Opened by geega almost 3 years ago
Labels: merge-only
#359 - FRW-332 Fix up PHPStan and typeless param.
Pull Request -
State: closed - Opened by dereuromark almost 3 years ago
Labels: bug, prio, merge-only
#358 - FRW-380: PHP 8.0-8.2 support checks
Issue -
State: closed - Opened by dereuromark almost 3 years ago
- 4 comments
Labels: improvement
#357 - Add missing elseif case.
Pull Request -
State: closed - Opened by dereuromark almost 3 years ago
Labels: bug, ready-for-release
#356 - Add new sniffers for strict_types declaration
Pull Request -
State: closed - Opened by DmytroKlymanSpryker almost 3 years ago
- 10 comments
#355 - Add "static analysis" Composer keyword
Pull Request -
State: closed - Opened by GaryJones almost 3 years ago
#354 - PHP 82
Pull Request -
State: closed - Opened by dereuromark almost 3 years ago
#353 - Add new trailing comma sniff
Pull Request -
State: closed - Opened by dereuromark almost 3 years ago
#352 - APPS-4764 Exclude Spryker generated files.
Pull Request -
State: closed - Opened by stereomon almost 3 years ago
- 2 comments
#351 - Adjust ruleset for Strict regarding PHP 7.4+
Pull Request -
State: closed - Opened by dereuromark almost 3 years ago
#350 - Fix up intersection declaration support.
Pull Request -
State: closed - Opened by dereuromark about 3 years ago
#349 - Support for nullable type notation
Issue -
State: open - Opened by hemberger about 3 years ago
- 3 comments
Labels: improvement
#348 - Improve bootstrap.php
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
#347 - Remove Composer tool installation related code
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
- 1 comment
#346 - Fix: Sort packages
Pull Request -
State: closed - Opened by localheinz about 3 years ago
- 1 comment
#345 - Enhancement: Allow installation of `slevomat/coding-standard:^8.0.1`
Pull Request -
State: closed - Opened by localheinz about 3 years ago
- 7 comments
Labels: improvement
#344 - Check PHP 8.2
Pull Request -
State: closed - Opened by dereuromark about 3 years ago
#343 - Fix order of CI steps
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
#342 - Change namespace of tests to Spryker\Test
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
#341 - Allow using cache
Issue -
State: closed - Opened by localheinz about 3 years ago
- 2 comments
Labels: improvement
#340 - Use PHPDoc license tag in *all* files
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
- 1 comment
#339 - Simply docs generator
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
- 2 comments
#338 - Suggest phpcodesniffer-composer-installer
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
- 2 comments
Labels: improvement
#337 - Obey PHPCSDevCS coding standard
Issue -
State: closed - Opened by szepeviktor about 3 years ago
- 3 comments
#336 - Remove stray/legacy .license file
Issue -
State: closed - Opened by szepeviktor about 3 years ago
- 9 comments
#335 - Add vendor name to tests
Issue -
State: closed - Opened by szepeviktor about 3 years ago
#334 - Remove Composer tool related code
Issue -
State: closed - Opened by szepeviktor about 3 years ago
- 3 comments
#333 - Improve code sniffer
Issue -
State: closed - Opened by szepeviktor about 3 years ago
#332 - Fix DisallowArrayTypeHintSyntaxSniff
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
- 4 comments
#331 - Check XML files against schema
Pull Request -
State: closed - Opened by szepeviktor about 3 years ago
- 2 comments
#330 - Read namespaces from spryker config
Pull Request -
State: open - Opened by alfredbez about 3 years ago
- 1 comment
Labels: improvement, prio
#329 - Sniff ordering causes side effect
Issue -
State: open - Opened by dereuromark about 3 years ago
- 1 comment
Labels: bug
#328 - Fix FQCN and whitespace issues.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, ready-for-release
#327 - Align class doc blocks
Issue -
State: closed - Opened by stereomon over 3 years ago
- 2 comments
Labels: bug
#326 - Fix use statement fixer
Issue -
State: closed - Opened by stereomon over 3 years ago
Labels: bug
#325 - Fix path detection for nonsplit
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, prio, merge-only
#324 - Add documentation for hasMethodAnnotation
Pull Request -
State: closed - Opened by alfredbez over 3 years ago
#323 - Fix getParentClassesFor method return type
Pull Request -
State: closed - Opened by alfredbez over 3 years ago
#322 - Fix method annotation sniff when using final classes
Pull Request -
State: closed - Opened by alfredbez over 3 years ago
Labels: bug
#321 - Check all parent classes in extendsAbstract(), not only one direct parent
Pull Request -
State: closed - Opened by alfredbez over 3 years ago
- 1 comment
#320 - Improve method annotation sniff
Pull Request -
State: closed - Opened by alfredbez over 3 years ago
- 6 comments
Labels: improvement
#319 - Fix newline issue on constants.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement
#318 - TE-10859 Fix false positive in use statement sniff.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, ready-for-release
#317 - Fix throw annotation sniff regarding static calls.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug
#316 - Wrong @throw annotation suggested
Issue -
State: closed - Opened by alfredbez over 3 years ago
- 5 comments
Labels: bug
#315 - TE-10552 Using generics for all collections including all objects
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
- 2 comments
Labels: ready-for-release, merge-only
#314 - Investigate list<type> sniff usage
Issue -
State: closed - Opened by dereuromark over 3 years ago
- 1 comment
Labels: improvement
#313 - Add DisallowTrailingCommaInSingleLine to avoid side effects.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, ready-for-release
#312 - No trailing comma in single line
Issue -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug
#311 - Allow all generics, now that PHPStorm does
Issue -
State: closed - Opened by dereuromark over 3 years ago
- 1 comment
Labels: improvement
#310 - Allow UselessParenthesesSniff with basic config
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement
#309 - Investigate UselessParenthesesSniff
Issue -
State: closed - Opened by dereuromark over 3 years ago
- 1 comment
Labels: improvement
#308 - TE-10370 Add DisallowCloakingCheck sniff.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement, ready-for-release
#307 - Fix up return tag sniff for generics.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, prio
#306 - Incorrect fix of nested nullable types
Issue -
State: closed - Opened by kkicheglovspryker over 3 years ago
- 1 comment
Labels: bug, prio
#305 - empty()/isset() only for possibly undefined keys
Issue -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement
#304 - Make sure FQCN adds no extra leading blackslash.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug
#303 - Fix inline docblock sniff.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
#302 - Fix throws annotation fixing.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
#301 - Remove silencing to fix up method signature whitespace
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug
#300 - Method signature whitespace
Issue -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, prio
#299 - Fix up more FQCN issues in docblocks.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
#298 - Add SlevomatCodingStandard.Commenting.InlineDocCommentDeclaration
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement, ready-for-release
#297 - Add Spryker.Commenting.DocBlockReturnNull
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
#296 - Add Assignment in Conditions sniff
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement
#295 - Move some internal sniffs.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
#294 - Add Prefer static over strict sniff.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement
#293 - Improve as per new generics. Add tests.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
#292 - Add more cleanup and add tests.
Pull Request -
State: closed - Opened by dereuromark over 3 years ago
Labels: bug, improvement, prio
#291 - bug in Spryker.Commenting.DocBlockVariableNullHintLast.WrongType
Issue -
State: closed - Opened by alfredbez over 3 years ago
- 3 comments
Labels: bug
#290 - SlevomatCodingStandard.ControlStructures.AssignmentInCondition
Issue -
State: closed - Opened by dereuromark over 3 years ago
Labels: improvement
#289 - Fix up duplicate removal.
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: improvement, ready-for-release
#288 - Collection fixes
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: major
#287 - Remove requirement to change complex object collections.
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: bug, prio, ready-for-release
#286 - Fix CS sniff for const spacing.
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: improvement, ready-for-release
#285 - Missing empty line after @api doc block is not detected
Issue -
State: closed - Opened by floriankraemer almost 4 years ago
Labels: improvement
#284 - Add critical BC sniff for core level.
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: improvement, prio, ready-for-release
#283 - PHP 74 property type null default
Issue -
State: closed - Opened by dereuromark almost 4 years ago
- 1 comment
Labels: improvement
#282 - Add missing sniffs that are already expected.
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: bug, improvement, ready-for-release, major
#281 - Add 7.4 sniffs.
Pull Request -
State: closed - Opened by dereuromark almost 4 years ago
Labels: improvement, ready-for-release
#280 - 7.4+ compatibility check
Issue -
State: closed - Opened by dereuromark almost 4 years ago
- 1 comment
Labels: improvement, prio
#279 - Indentation and whitespace issues
Issue -
State: closed - Opened by dereuromark almost 4 years ago
- 1 comment
Labels: bug