Skip to content

Combine consecutives issets#6910

Merged
Ocramius merged 1 commit intodoctrine:developfrom
carusogabriel:remove_consecutive_issets
Dec 19, 2017
Merged

Combine consecutives issets#6910
Ocramius merged 1 commit intodoctrine:developfrom
carusogabriel:remove_consecutive_issets

Conversation

@carusogabriel
Copy link
Copy Markdown
Contributor

isset allow us to pass multiple values to check.

PHP-CS-Fixer helped me on that.

Copy link
Copy Markdown
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Ocramius Ocramius added this to the 3.0 milestone Dec 19, 2017
@Ocramius Ocramius merged commit 9812212 into doctrine:develop Dec 19, 2017
@Ocramius Ocramius self-assigned this Dec 19, 2017
@carusogabriel carusogabriel deleted the remove_consecutive_issets branch December 19, 2017 17:24
@greg0ire greg0ire removed this from the 3.0.0 milestone Jun 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants