Skip to content

Sync FindExtendedClassname() method with upstream#507

Merged
wimg merged 1 commit intomasterfrom
feature/sync-in-upstream-change-extends
Aug 7, 2017
Merged

Sync FindExtendedClassname() method with upstream#507
wimg merged 1 commit intomasterfrom
feature/sync-in-upstream-change-extends

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Aug 7, 2017

Synced in the changes as were just merged from squizlabs/PHP_CodeSniffer/pull/1473

This allows the method to also detect the "Extended Classname" for interfaces extending other interfaces.

Adjusted the related getFQExtendedClassName() method accordingly and added unit tests for this change.

The change to the FindExtendedClassname() method is tested upstream.

@jrfnl jrfnl requested a review from wimg August 7, 2017 01:01
@jrfnl jrfnl force-pushed the feature/sync-in-upstream-change-extends branch from 80e9aac to a304da3 Compare August 7, 2017 01:38
Synced in the changes as were just merged from squizlabs/PHP_CodeSniffer/pull/ 1473

This allows the method to also detect the "Extended Classname" for interfaces extending other interfaces.

Adjusted the related `getFQExtendedClassName()` method accordingly and added unit tests for this change.

The change to the `FindExtendedClassname()` method is tested upstream.
@jrfnl jrfnl force-pushed the feature/sync-in-upstream-change-extends branch from a304da3 to 2be6838 Compare August 7, 2017 01:57
@wimg wimg added this to the 8.0.2 milestone Aug 7, 2017
@wimg wimg merged commit e3a7ae6 into master Aug 7, 2017
@wimg wimg deleted the feature/sync-in-upstream-change-extends branch August 7, 2017 19:44
@jrfnl jrfnl modified the milestones: 8.0.2, 8.1.0 Nov 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants