search

davedevelopment / phpmig

Simple migrations system for php
Other
569 stars 92 forks source link

remove unused imports, fix Symfony 5 compatibility, update docblocks #150

Closed IonBazan closed 3 years ago

IonBazan commented 3 years ago

Started with removing unused imports and fixing Symfony 5 compatibility (Commands should always return int), ended up fixing code style (missing typehints as we use PHP 7.1+, @inheritDoc instead of copy-pasted outdated phpdocs, short array-syntax, etc).

IonBazan commented 3 years ago

Travis build is now fixed too. @davedevelopment you may want to reinstall Travis integration, as the repository is still using the legacy one.

davedevelopment commented 3 years ago

Thank you, I'll take a note to try and look at the travis integration, but I'm not very active on this project and it might take a while :)

IonBazan commented 3 years ago

Thanks for merging and maintaining the project!