Thanks to visit codestin.com
Credit goes to github.com

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

.php_cs - sort fixers#7011

Merged
Ocramius merged 1 commit into
zendframework:developfrom
keradus:php_cs-sort_fixers
Dec 11, 2014
Merged

.php_cs - sort fixers#7011
Ocramius merged 1 commit into
zendframework:developfrom
keradus:php_cs-sort_fixers

Conversation

@keradus
Copy link
Copy Markdown
Contributor

@keradus keradus commented Dec 11, 2014

No description provided.

Comment thread .php_cs
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we keep the diff minimal? :|

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am afraid that I do not understand what do you mean by that.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah, nvm, just saw that you only sorted them. That's fine as long as there were no functional changes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes. just want the list to be clean when adding new items in it ;)
For next PRs to be minimal ;)

Should I do anything else to remove that awaiting author updates label ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'll do that

@localheinz
Copy link
Copy Markdown
Member

👍

Ocramius added a commit that referenced this pull request Dec 11, 2014
@Ocramius Ocramius merged commit 40aa847 into zendframework:develop Dec 11, 2014
@Ocramius
Copy link
Copy Markdown
Member

@keradus merged, thanks!

@keradus keradus deleted the php_cs-sort_fixers branch December 11, 2014 18:18
weierophinney pushed a commit to zendframework/zend-i18n-resources that referenced this pull request May 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants