Skip to content
This repository has been archived by the owner on Nov 15, 2022. It is now read-only.

dependencies update #97

Merged
merged 1 commit into from
Jan 9, 2018
Merged

Conversation

aldas
Copy link
Contributor

@aldas aldas commented Jan 5, 2018

This addresses #96

NB: CC_TEST_REPORTER_ID value in .travis.yml should be changed to hash that is in codeclimate for this project. It can be found under 'Settings > Test Coverage > 'TEST REPORTER ID'. I used same value that was old reporter used but I'm not sure if it is valid for new reporter.

* minor updates related to code style
* clean dev dependencies of symphony references
* use phpunit 5.7 and it does not support php 5.5 anymore (it is EOL anyway) remove 5.5 from travis builds
* update codeclimate to use newer reporter as old will be deprecated
@aldas
Copy link
Contributor Author

aldas commented Jan 7, 2018

travis build should be rerun as it had network timeout for one of the builds (https://travis-ci.org/influxdata/influxdb-php/jobs/325597989)

about CC_TEST_REPORTER_ID I think it could (should probably) configured as travis env variable in travis settings https://docs.travis-ci.com/user/environment-variables/.

@gianarb
Copy link
Contributor

gianarb commented Jan 9, 2018

All good thanks!

@gianarb gianarb merged commit a66475b into influxdata:master Jan 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants