Skip to content
This repository was archived by the owner on May 1, 2019. It is now read-only.

Conversation

@GeeH
Copy link
Contributor

@GeeH GeeH commented Apr 27, 2015

No description provided.

@GeeH GeeH force-pushed the feature/add-Scrutinizer branch from 214d14e to 88e5ef2 Compare April 27, 2015 09:26
Copy link
Member

Choose a reason for hiding this comment

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

@GeeH

We don't have docs or examples directories, do we?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no, it's a standard config I use across projects and does no harm

@localheinz
Copy link
Member

@GeeH

Something's still wrong with the Scrutinizer setup, can you have a look?

Copy link
Member

Choose a reason for hiding this comment

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

@GeeH

I guess we should not run php-cs-fixer on Travis, then?

Copy link
Member

Choose a reason for hiding this comment

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

Scrutinizer is not going to fail - we want a failure on CS issues

Copy link
Member

Choose a reason for hiding this comment

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

@Maks3w
Copy link
Member

Maks3w commented May 31, 2015

This proposal should be raised to crteam ML

@GeeH GeeH closed this Aug 2, 2016
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.

4 participants