Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Scrutinizer #132

Merged
merged 7 commits into from Feb 5, 2019
Merged

Remove Scrutinizer #132

merged 7 commits into from Feb 5, 2019

Conversation

@Jean85
Copy link
Member

Jean85 commented Feb 5, 2019

Scrutinizer build started failing because it detects PHPUnit configs in the stub folder.

Since the introduction of PHPStan, it's no longer useful: CS style is out of sync, tests are run elsewhere, and sniffs are bad.

I'll fix the Coveralls reporting to not lose coverage report, which was the only useful thing provided by Scrutinizer.

@Jean85 Jean85 added this to the Paraunit 1.0 milestone Feb 5, 2019
@Jean85 Jean85 self-assigned this Feb 5, 2019
Jean85 added 6 commits Feb 5, 2019
@Jean85 Jean85 merged commit 87f011e into master Feb 5, 2019
2 of 4 checks passed
2 of 4 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
codeclimate All good!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@Jean85 Jean85 deleted the disable-scrutinizer branch Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.