Skip to content

Conversation

@dbu
Copy link
Contributor

@dbu dbu commented Jan 14, 2022

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets -
Documentation -
License MIT

What's in this PR?

Run phpstan on the codebase

@dbu dbu force-pushed the static-analysis branch from 03aa3ff to eb2d254 Compare January 14, 2022 16:23
$this->stopwatch->start($eventName, self::CATEGORY);

return $next($request)->then(function (ResponseInterface $response) use ($eventName) {
$this->stopwatch->stop($eventName, self::CATEGORY);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Stopwatch::stop never expected the second argument, only start has that.

@dbu dbu merged commit 0daec9b into master Jan 14, 2022
@dbu dbu deleted the static-analysis branch January 14, 2022 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants