-
-
Notifications
You must be signed in to change notification settings - Fork 14
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.
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
Added types #29
Added types #29
Conversation
64b30b3
to
fc3ee63
Compare
@pionl Rebased |
|
||
$containerConfigurator->parameters() | ||
->set(Option::PARALLEL, false); | ||
$containerConfigurator->cacheDirectory('/tmp/ecs'); |
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.
je tohle safe? Ne na každém operačím systému to bude fungovat ne?
{ | ||
return $this->customField; | ||
} | ||
|
||
public function toArray(): array | ||
{ | ||
return $this->customField->jsonSerialize(); | ||
return $this->customField !== null ? $this->customField->jsonSerialize() : []; |
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.
$this->customField?->jsonSerialize() ?? []
Použít tuto syntaxi?
*/ | ||
public function byId($byId) | ||
public function byId($byId): self |
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.
přidat string|int?
Mergnu, můžeme řešiv dalším MR |
This PR is based on #28 and adds type declaration wherever available and removes few unnecessary complications (like global helpers)