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

traits require #50

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

traits require #50

wants to merge 18 commits into from

Conversation

Hidanio
Copy link
Contributor

@Hidanio Hidanio commented Aug 20, 2024

No description provided.

Hidanio and others added 17 commits September 28, 2023 18:13
@Hidanio
Added new cases for tests
1a2d4db
@Hidanio
added traits nesting handler
67c4249
@Hidanio
added condition for situation where PhpUse is use as import (not trait)
9190dc7 
added support for trait import/required inspection
@Hidanio
added more logic for 'required' traits
8dd121c 
added new tests for inspections
@Hidanio
added integration test
e9d5178
@Hidanio
refactored logic for traits in InternalSymbolUsageInspection
cf339a1
@Hidanio
added integration test for alone trait (not nesting)
cbe0a2b
@Hidanio
moved test 006 to 101 because the bug is fixed and this is expected b…
ecb2097 
…ehavior
@Hidanio
Rewrote logic for traits collecting and inspection
405e3ae
@Hidanio
fixed duplicate bug in trait inspection apply
e89a324 
added new test from kphp
moved
@Hidanio
rewrote collecting modulite dependencies logic for traits
3d1cc3d
@Hidanio
fixed types from params, return
0d67831
@Hidanio
fixed type inference from parameters, added deep recursion
3e81ffb
@Hidanio
refactoring
e345caa
@Hidanio
merged logic about trait in collector
d933bdf
@Hidanio
Expanded inspection logic
c0c17c1
@Hidanio
Merge branch 'master' into hidanjkezor/traits_require_feature
ff5c1ae
@Hidanio Hidanio self-assigned this Aug 20, 2024
@Hidanio Hidanio added the enhancement New feature or request label Aug 20, 2024
@Hidanio Hidanio requested a review from Danil42Russia August 20, 2024 11:39
@Hidanio Hidanio changed the title Traits require traits require Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Danil42Russia Danil42Russia Awaiting requested review from Danil42Russia

At least 1 approving review is required to merge this pull request.

Assignees

@Hidanio Hidanio

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

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