Update default to 8.7 - Githubissues

LionsAd / drupal_ti

Drupal - Travis Integration

82 stars 37 forks source link

Update default to 8.7 #122

Closed taz77 closed 5 years ago

taz77 commented 5 years ago

Drupal 8.1 was deprecated quite some time ago and has security vulnerabilities and should not be used. There is also a lot of bug fixes for testing.

coveralls commented 5 years ago

Coverage remained the same at 100.0% when pulling 9ab6dba6e006114542c9b2a7681c8555ac23f88c on Fastglass-LLC:update-drupal8-default into e66a53fc6225869a70188bdb2b90c8f4ae4cac6e on LionsAd:master.

WengerK commented 5 years ago

I'm not pretty sure how this change may affect users of drupal_ti.

People can change to the desired version using the DRUPAL_TI_CORE_BRANCH but people who never use it will see there build crash as we force to 8.7.x instead of 8.1.x

Do you have any more arguments to push this commit on master ?

taz77 commented 5 years ago

If they are using Drupal 8.1.x they have problems far worse than Drupal_ti changes. The security holes in those versions are rather severe.