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.

Fix several cs#2306

Merged
weierophinney merged 5 commits into
zendframework:masterfrom
blanchonvincent:fix-several-cs
Sep 13, 2012
Merged

Fix several cs#2306
weierophinney merged 5 commits into
zendframework:masterfrom
blanchonvincent:fix-several-cs

Conversation

@blanchonvincent
Copy link
Copy Markdown
Contributor

Fix several coding standard to keep new 2.0.0 more clean :)

fix "use" and multiple spaces
add fix cs with space after comma
add space with string concatenation
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.

space before 1

@blanchonvincent
Copy link
Copy Markdown
Contributor Author

Maks3w, add new commit to add CS with your comment.

@blanchonvincent
Copy link
Copy Markdown
Contributor Author

New fixs and fix with your comment.

fix whitespace 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.

This is not doing what you think it's doing, and causes a test failure.

weierophinney added a commit that referenced this pull request Sep 13, 2012
- CS does not apply within strings
@weierophinney weierophinney merged commit 10ee647 into zendframework:master Sep 13, 2012
@weierophinney
Copy link
Copy Markdown
Member

Fixed the XmlRpc testing issue, and merged to release and master.

@ghost ghost assigned weierophinney Sep 13, 2012
@blanchonvincent
Copy link
Copy Markdown
Contributor Author

oh, don't see, sorry!

weierophinney added a commit to zendframework/zend-xmlrpc that referenced this pull request May 15, 2015
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.

3 participants