Skip to content

Update build to add PHPCS and PHPStan#130

Merged
lcobucci merged 6 commits intomasterfrom
update-build
Aug 30, 2017
Merged

Update build to add PHPCS and PHPStan#130
lcobucci merged 6 commits intomasterfrom
update-build

Conversation

@lcobucci
Copy link
Copy Markdown
Member

No description provided.

@lcobucci lcobucci added this to the 1.6.0 milestone Aug 29, 2017
@lcobucci lcobucci self-assigned this Aug 29, 2017
@lcobucci lcobucci requested a review from Ocramius August 29, 2017 16:35
phpcs.xml.dist Outdated
@@ -0,0 +1,22 @@
<?xml version="1.0"?>
<ruleset name="Coding Standards for doctrine">
<description>Coding Standards for doctrine.</description>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should probably be "Coding Standard for Doctrine Collections." (and same for ruleset name)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

You're right, I was too lazy to change it (and I probably didn't change on the other projects as well - shame on me!).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've just found out that the name attribute and the description element are useless here. So I'll just kill them

colors="true"
verbose="true"
beStrictAboutOutputDuringTests="true"
beStrictAboutTestsThatDoNotTestAnything="true"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why was this removed?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've tried explained on the commit message (cb7f39d) but the reason is that we were just repeating the default configuration, which is kind of redundant.

Copy link
Copy Markdown
Member

@alcaeus alcaeus left a comment

Choose a reason for hiding this comment

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

👍

@lcobucci lcobucci merged commit 42c4039 into master Aug 30, 2017
@lcobucci lcobucci deleted the update-build branch August 30, 2017 09:16
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