Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BetterNodeFinder + NodeRemover added #113

Merged
merged 33 commits into from
Oct 28, 2017
Merged

BetterNodeFinder + NodeRemover added #113

merged 33 commits into from
Oct 28, 2017

Conversation

TomasVotruba
Copy link
Member

Ref #111

@TomasVotruba
fix Name node resolving
78903bc
@TomasVotruba
fix Name node resolving
b39efbf
@TomasVotruba
raw init of ExpressionRemover
91559ab
@TomasVotruba
add BetterNodeFinder, change ExpressionRemover to NodeRemover
4cc5e84
@TomasVotruba
make test pass
e1efffe
@TomasVotruba
insurance
aa67e22
@TomasVotruba
AbstractRector: drop decouple shouldRemoverNode logic
109da12
@TomasVotruba
[NodeTraverserQueue] add BetterNodeFinder tests
5d1b918
@TomasVotruba
add BetterNodeFinder to AbstractRector
5a18744
@TomasVotruba
drop dead code
fbe86bf
@TomasVotruba
use BetterNodeFinder instead of Printer magic
af68eaf
@TomasVotruba
misc
5bfa37d
@TomasVotruba
misc
8451443
@TomasVotruba
add segfault test
779966b
@TomasVotruba
fix segfault in PropertyFetchTypeResolver
f015173
@TomasVotruba
misc
304e22d
@TomasVotruba
details
31da6ea
@TomasVotruba
[NodeTraverserQueue] add test for weird cloning use case
3d7e03a
@TomasVotruba
add failing test for CloningVisitor
d59ffe2
@TomasVotruba
add IdentifierRector test
0d6abda
@TomasVotruba
extend IdentifierRector test
200ab47
@TomasVotruba
IdentifierRector: run just once
4b244db
@TomasVotruba
ClassReplacerRector test
70bd97c
@TomasVotruba
fix cs
9979677
@TomasVotruba
fixing PerNodeTypeResolver for property access
be26c7e
@TomasVotruba
[ReflectionDocBlock] make NamespaceAnalyzer work with use aliases
b240ead
@TomasVotruba
[PropertyFetchTypeResolver] add object property reflection
9f9ed78
@TomasVotruba
[NodeTypeResolver] deal with nested property
ea2a918
@TomasVotruba
make it run for nested array property access
46c2009
@TomasVotruba
minimize code for failing clone of NodeTraverserQueue
47a5625
@TomasVotruba TomasVotruba merged commit 09b8642 into master Oct 28, 2017
@TomasVotruba TomasVotruba deleted the node-finder-expression-remover branch October 28, 2017 15:01
@TomasVotruba TomasVotruba mentioned this pull request Oct 29, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@TomasVotruba