Skip to content
This repository was archived by the owner on Jun 29, 2022. It is now read-only.

Not found handler tests #134

Merged
merged 2 commits into from
Oct 30, 2019
Merged

Not found handler tests #134

merged 2 commits into from
Oct 30, 2019

Conversation

romkatsu
Copy link
Member

@romkatsu romkatsu commented Oct 29, 2019

Q A
Is bugfix?
New feature?
Breaks BC?
Tests pass? ✔️
Fixed issues #105

@samdark samdark added status:code review The pull request needs review. type:test Test labels Oct 29, 2019
Copy link
Member

@samdark samdark left a comment

Choose a reason for hiding this comment

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

Let's adjust test names and merge it 👍

@samdark samdark merged commit 4214e2f into yiisoft:master Oct 30, 2019
@samdark
Copy link
Member

samdark commented Oct 30, 2019

Merged. Thanks!

devanych pushed a commit that referenced this pull request Feb 1, 2021
devanych pushed a commit that referenced this pull request Feb 1, 2021
devanych pushed a commit that referenced this pull request Feb 1, 2021
devanych pushed a commit that referenced this pull request Feb 1, 2021
devanych pushed a commit that referenced this pull request Feb 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:code review The pull request needs review. type:test Test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants