Skip to content
This repository was archived by the owner on Aug 16, 2020. It is now read-only.

Reviewdogに指摘されるブランチ#1

Open
kadoyau wants to merge 2 commits into
masterfrom
circleci-test
Open

Reviewdogに指摘されるブランチ#1
kadoyau wants to merge 2 commits into
masterfrom
circleci-test

Conversation

@kadoyau
Copy link
Copy Markdown
Owner

@kadoyau kadoyau commented Dec 7, 2019

reviewdogにレビューをされるとどうなるのかわかるためのサンプルPRです


public function thisFunctionShouldBeWatchedByDog(): string
{
return $this->age;
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[phpstan] reported by reviewdog 🐶
Method Acme\Domain\VTuber::thisFunctionShouldBeWatchedByDog() should return string but returns int.


public function thisFunctionShouldBeWatchedByDog2(): string
{
return $this->age;
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[phpstan] reported by reviewdog 🐶
Method Acme\Domain\VTuber::thisFunctionShouldBeWatchedByDog2() should return string but returns int.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant