Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

New coding standard #22

Closed
wants to merge 3 commits into from
Closed

New coding standard #22

wants to merge 3 commits into from

Conversation

Koopzington
Copy link
Contributor

Usage of the newly introduced zendframework/zend-coding-standard for CS checks.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5e68170 on Koopzington:new-coding-standard into e882abb on zendframework:master.

@weierophinney weierophinney self-assigned this Dec 13, 2016
weierophinney added a commit that referenced this pull request Dec 13, 2016
weierophinney added a commit that referenced this pull request Dec 13, 2016
@weierophinney
Copy link
Member

Merged to develop for release with 1.3.0. Thanks, @Koopzington .

weierophinney added a commit that referenced this pull request Dec 13, 2016
Forward ported changes from:

- #22
- #23
- #24
- #25

The changes in #23 allowed removing the changes introduced in #21, as
the route path is now accessible via the composed `Route` instance
(assuming that `fromRoute()` was used to create the `RouteResult`
instance).

Conflicts:
	src/RouteResultObserverInterface.php
	src/RouteResultSubjectInterface.php
	test/RouteResultTest.php
@Koopzington Koopzington deleted the new-coding-standard branch December 19, 2016 22:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants