-
Notifications
You must be signed in to change notification settings - Fork 87
Add php 7.1 type hints #247
base: develop
Are you sure you want to change the base?
Conversation
|
@snapshotpl this needs a rebase, sorry :-\ |
|
@Ocramius done! |
| * {@inheritdoc} | ||
| */ | ||
| public function canCreate(\Psr\Container\ContainerInterface $container, $requestedName) | ||
| public function canCreate(ContainerInterface $container, string $requestedName): bool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any plans to release this feature of coding standard?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add your question to the related PR. Thanks!
| * {@inheritDoc} | ||
| */ | ||
| public function canCreate(ContainerInterface $container, $requestedName) | ||
| public function canCreate(ContainerInterface $container, string $requestedName): bool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here…
| private function resolveParameterWithoutConfigService( | ||
| ContainerInterface $container, | ||
| string $requestedName | ||
| ): callable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here and so on…
|
This repository has been closed and moved to laminas/laminas-servicemanager; a new issue has been opened at laminas/laminas-servicemanager#14. |
|
This repository has been moved to laminas/laminas-servicemanager. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
No description provided.