rectorphp/rector-src issues and pull requests

#4763 - [Cleanup] Remove unused easy-ci config

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4762 - Make indent size and char fetchable

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4761 - [DX] Deprecate NonPhpRectorInterface, the only rule and its file processor, to make Rector handle exlusively PHP

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

#4760 - Remove Dumper::dump() in print_node function

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#4759 - [DX] deprecated build shipped debug fuctions, allow using custom debug package instead; add local d() and dd() functions

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 2 comments

#4758 - [Php80] Handle crash Property ReflectionEnum::$betterReflectionClass does not exist on AddParamBasedOnParentClassMethodRector on Enum usage

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 4 comments

#4757 - [DX] Remove complementary interface, as not practical

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4756 - cleanup phpstan errors

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4755 - [Renaming] Deprecate PseudoNamespaceToNamespaceRector as too dynamic and unreliable, use the RenameClassRector instead

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4754 - [TypeDeclaration] Use native type detection instead of docblock on AssignToPropertyTypeInferer for TypedPropertyFromAssignsRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 5 comments

#4753 - Cleanup test fixture

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4752 - Use direct VoidType definition on Return_ expr is null on ReturnedNodesReturnTypeInfererTypeInferer

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

#4751 - [TypeDeclaration] Skip void type on caller on ReturnTypeFromStrictTypedCallRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#4750 - [TypeDeclaration] Skip return has Expr even void result on AddVoidReturnTypeWhereNoReturnRector

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

#4749 - [DX] Cleanup and simplify class naming tests

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4748 - misc

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4747 - cleanup

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4746 - Remove Php80NestedAttributesRector test as coverage in PHP 8 test

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4745 - [Renaming] Remove callables from class renames to avoid missed rename bugs, better handle with PHSPtan and custom rule

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4744 - [DX] Less rename class tests to keep readable and unique

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4743 - [DX] Avoid supporting edge-case multi-layered comments, must be covered in php-parser itself

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

#4742 - [DX] Add simple parmater provider hash

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4741 - [DX] Reduce getFilePath() call on AbstractRector

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

#4740 - [DX] Cleanup covered multline doc printer test

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4739 - cleanup spaces

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4738 - [DX] Keep parameter pure file paths

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4737 - [DX] Ensure file processors are unique to avoid double run

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4736 - [Logging] Add RectorOutput logging service

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#4735 - Make relativeFilePathFromDirectory() private as used only locally

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4734 - [DX] Add ProcessConfigureDecorator to avoid superfluous composition

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4733 - [DX] Move RectifiedAnalyzer to Skipper service

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

#4732 - Remove appendArgs() method on AbstractRector

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

#4731 - [Php70] Remove appendArgs() usage on CallUserMethodRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#4730 - [WIP] Symplify removing unreachable node

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 11 comments

#4729 - [DX] Remove AddDefaultValueForUndefinedVariableRector as can generate broken code and depends on context

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4728 - [DX] Validate rules no longer exists on $rectorConfig->skip()

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#4727 - [TypeDeclaration] Allow change multiple methods on AddParamTypeDeclarationRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#4726 - Reduce private accessor usage on get parent class name not autoload

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

#4725 - [DX] Laravel container + test case improvements

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4724 - [DX] Remove autoimport from unrelated tests, handled in importing

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4723 - [TypeDeclaration] Skip non void or never already return typed on ReturnNeverTypeRector

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

#4714 - Update composer.json

Pull Request - State: closed - Opened by bbrala over 1 year ago - 13 comments

#4698 - [DI] Switch container to Laravel - final step 🥳

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 5 comments

#4645 - MyCLabs enum equals + keeping methods

Pull Request - State: closed - Opened by K0nias over 1 year ago - 8 comments

#4641 - Create do_not_duplicate_annotations.php.inc

Pull Request - State: closed - Opened by mkrauss over 1 year ago - 6 comments

#4577 - [WIP] Try phpstan printer

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

#4573 - Make use of toPhpDocNode() - take #10

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 2 comments

#4560 - try phpdocnode on mappers

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#4552 - implemented TypedPropertyFromStrictConstructorReadonlyClassRector

Pull Request - State: closed - Opened by staabm over 1 year ago - 4 comments

#4518 - Fix code sample for AttributeKeyToClassConstFetchRector

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

#4492 - [Php80] Handle invalid change on extends PDO class on AddParamBasedOnParentClassMethodRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 16 comments

#4482 - RemoveUselessReturnTagRector doesn't recognize tag comments on a new line given a PHP core type hint

Pull Request - State: closed - Opened by TravisCarden over 1 year ago - 2 comments

#4477 - [WIP] Mimic 4474 for previous 0.17.2

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 2 comments

#4448 - [MysqlToMysqli] Deprecate set, as mostly breakingcode, rather handle upgrade manually

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 4 comments

#4442 - [PHP 8] Resources object upgrade - remove 2 rules as the upgrade require wholesome manual work

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago - 2 comments

#4411 - [DX] Remove provideParameter() method

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

#4224 - fix for php 8.3

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

#4191 - [Core] Decorate all below nodes only on usage of DONT_TRAVERSE_CHILDREN or DONT_TRAVERSE_CURRENT_AND_CHILDREN

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 8 comments

#4146 - Drop AttributeKey::SCOPE in NodeTypeResolver's

Pull Request - State: closed - Opened by staabm over 1 year ago

#4135 - Deprecate AttributeKey::SCOPE

Pull Request - State: closed - Opened by staabm over 1 year ago

#4074 - [NodeTypeResolver] Handle crash after next exit() on no namespaced code after removal next attribute

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 6 comments

#4068 - Drop AttributeKey::SCOPE in ReflectionResolver

Pull Request - State: closed - Opened by staabm over 1 year ago - 2 comments

#3933 - [DeadCode][CodeQuality] Fix infinite loop on RemoveUnusedVariableAssignRector+SimplifyIfElseToTernaryRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 4 comments

#3932 - Remove PARENT_NODE from RemoveEmptyMethodCallRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3931 - [CodingStyle] Skip jump change variable name on CatchExceptionNameMatchingTypeRector

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 2 comments

#3930 - Remove array function from RemoveEmptyMethodCallRector as too narrow

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3929 - Remove PARENT_NODE from CatchExceptionNameMatchingTypeRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3928 - Remove PARENT_NODE from StringClassNameToClassConstantRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3927 - Remove AddPregQuoteDelimiterRector as very narrow area and has not context awareness, better handle manually

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3926 - [DeadCode] Remove RemoveDuplicatedIfReturnRector as can lead to unexpeced consequences, better use static analysis and contextual handling

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3925 - Fix is assigned to NodeVisitor, remove PARENT_NODE from ExplicitMethodCallOverMagicGetSetRector

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

#3924 - Remove PARENT_NODE from CountOnNullRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3923 - Fix `PrivatesAccessor: use selectFromArgs() instead`

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

#3922 - [DX] Deprecate PARENT_NODE attribute, allow return of NodeTraverser::* in refactor() method

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3921 - [DX] Remove unused RESOLVED_NAME attribute, as now php-parser uses namespacedName property

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3920 - [ci] split rules tests to code quality, type declaration and dead-code to speed up

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

#3919 - try removing resolved name attribute, as namespaced name const

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3918 - [Traverser] Using bool value for assigned_to attribute on variable assign

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

#3917 - [DeadCode] Remove next attribute on RemoveUnusedVariableAssignRector

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

#3916 - Remove ChangeOrIfReturnToEarlyReturnRector as makes code harder to read by duplication, rather use PHSPtan and extract method

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3915 - Remove NEXT_NODE from SimplifyIfReturnBoolRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3914 - Remove NEXT_NODE from ChangeIfElseValueAssignToEarlyReturnRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3913 - Remove NEXT_NODE dependency in ChangeAndIfToEarlyReturnRector, narrow to only closed scope function likes

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3912 - [Parser] Fix Expr on edge Stmt start/end token detection on BetterNodeFinder

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3911 - Revert tweak start token pos compare

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3910 - [automated] Apply Coding Standard

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3909 - [automated] Re-Generate Nodes/Rectors Documentation

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3908 - [automated] Re-Generate preload.php

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3907 - [Parser] Cover faster edge Expr in first token or end token of Stmt on BetterNodeFinder

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3906 - [Parser] Clean up BetterNodeFinder::findFirstInTopLevelStmtsAware() param type

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3905 - [Parser] Allow resolve next/prev stmt inside ClassLike or Declare_ on BetterNodeFinder

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3904 - [Traverser] Clean up nullable stmt_key check on BetterNodeFinder::resolveNeighborNextStmt() take 2

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

#3903 - [Traverser] Clean up: Remove afterTraverse() on StmtKeyNodeVisitor

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3902 - [Traverser] Clean up nullable stmt_key check on BetterNodeFinder::resolveNeighborNextStmt()

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

#3901 - [Traverser] Clean up FileWithoutNamespace check on StmtKeyNodeVisitor

Pull Request - State: closed - Opened by samsonasik over 1 year ago

#3900 - [1.0] Replace NodeConnectingVisitor with ParentConnectingVisitor

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 8 comments

#3899 - [Parser] Fix key negative on BetterNodeFinder locate stmt on FileWithoutNamespace lookup Declare_

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

#3898 - [TypeDeclaration] Add BoolReturnTypeFromStrictScalarReturnsRector

Pull Request - State: closed - Opened by TomasVotruba over 1 year ago

#3897 - [Parser] Handle just removed stmt on BetterNodeFinder::resolveNodeFromFile() as fall for get next stmt

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

#3896 - [1.0] Add public method BetterNodeFinder::resolvePreviousNode() and BetterNodeFinder::resolveNextNode()

Pull Request - State: closed - Opened by samsonasik over 1 year ago - 9 comments

GitHub / rectorphp/rector-src issues and pull requests