★ wanayoo — archive 1999 https://github.com/phpDocumentor/ReflectionDocBlock/pull/139Nouvelle recherche | Portail wanayoo
Skip to content

/ ReflectionDocBlock

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

Jump to bottom

Add PHPStan and bump to max level #139

Merged
merged 10 commits into from Dec 2, 2017
Merged

Add PHPStan and bump to max level #139

merged 10 commits into from Dec 2, 2017

Conversation

@TomasVotruba
Copy link
Contributor

@TomasVotruba TomasVotruba commented Nov 30, 2017 •
edited

This should keep code without type errors as much as possible
TomasVotruba added 10 commits Nov 30, 2017
@TomasVotruba
composer: put require- together
4e49166
@TomasVotruba
add phpstan
d101615
@TomasVotruba
add phpstan to travis
3b0714e
@TomasVotruba
phpstan fixes
efe1aed
@TomasVotruba
phpstan fixes
e89b479
@TomasVotruba
phpstan fixes
7657dd3
@TomasVotruba
drop null defaults since every property is null by default
924b418
@TomasVotruba
DescriptionFactory: fix parse nullable type
e373b55
@TomasVotruba
travis: add phpstan max
Loading status checks…
db5883a
@TomasVotruba
travis: fix phpstan config
Loading status checks…
4e7e1e3
@TomasVotruba
Copy link
Contributor Author

@TomasVotruba TomasVotruba commented Nov 30, 2017

CI fixed 👍
@jaapio jaapio merged commit bab8870 into phpDocumentor:master Dec 2, 2017
3 checks passed
3 checks passed
Scrutinizer No new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 93.878%
Details
@jaapio
Copy link
Member

@jaapio jaapio commented Dec 2, 2017

Thanks!!!
@TomasVotruba
Copy link
Contributor Author

@TomasVotruba TomasVotruba commented Dec 2, 2017

You're welcome
@TomasVotruba TomasVotruba deleted the TomasVotruba:phpstan branch Dec 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@TomasVotruba @jaapio
You can’t perform that action at this time.