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

Run linter #109

Merged
merged 6 commits into from Mar 6, 2018
Merged

Run linter #109

merged 6 commits into from Mar 6, 2018

Conversation

dailycoding
Copy link
Contributor

PHP linter is not working with older version of PHP (< v5.5)

@f2prateek
Copy link
Contributor

Great, did the linter catch any errors?

@dailycoding
Copy link
Contributor Author

There's no error now.
If any error occurs, TravisCI testing will be failed.

@f2prateek f2prateek merged commit fcee5c2 into master Mar 6, 2018
@f2prateek f2prateek deleted the run_linter branch March 6, 2018 21:49
f2prateek pushed a commit that referenced this pull request Mar 6, 2018
* Enable code coverage and output report to coverage_report folder

* Remove --colors option in .travis.yml which is enabled in phpunit.xml

* Upload coverage result to codecov.io

* Add Linter to CI (#109)

* Run php linter before unit testing

* Install phplint before execute scripts

* Execute phplint when the php version is above or equal 5.5

* Run composer install when the php version is 5.5 or later

* Fix bash script error

* Fix bash script error in conditional expression

* Enable code coverage and output report to coverage_report folder

* Remove --colors option in .travis.yml which is enabled in phpunit.xml

* Upload coverage result to codecov.io
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants