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

[Fix] Silence warnings #253

Merged

Conversation

@olivernybroe
Copy link
Collaborator

olivernybroe commented Aug 21, 2019

Q A
Bug fix? yes
Fixed tickets #59

This PR removes the warnings thrown by PHP when a Sniff hits invalid code.

src/Domain/File.php Show resolved Hide resolved
@nunomaduro nunomaduro merged commit cafdb43 into nunomaduro:master Aug 22, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nunomaduro

This comment has been minimized.

Copy link
Owner

nunomaduro commented Aug 22, 2019

Amazing pull request, thank you for this!

olivernybroe added a commit to olivernybroe/phpinsights that referenced this pull request Aug 22, 2019
* Fix: When getting tokens, silence php warnings in case file is corrupt

* Fix: Silence sniff processor php warnings

* Style: Exclude silenced errors file from sniff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.