-
Notifications
You must be signed in to change notification settings - Fork 132
Conversation
…ypes. This is required starting with Drupal 8.7. Ref. https://www.drupal.org/node/2949023
…ecated in Drupal 8.7. Ref. https://www.drupal.org/node/3003360
…initions` in Drupal 8.7. Ref. https://www.drupal.org/node/3016699 This breaks backwards compatibility with Drupal 8.6 which does not yet support the `context_definitions` annotation.
2ac1476
to
4ca9ec7
Compare
There were a whole bunch of deprecation warnings when upgrading to 8.7. Most of them were pretty easy to fix since they link to change records with relevant information. One of the changes is incompatible with Drupal 8.6 and throws a deprecation warning on 8.7: https://www.drupal.org/node/3016699 We could exclude this deprecation warning from being checked, or we can declare a dependency on 8.7. I have opted for the latter, since this is consistent with how we did previous releases for OG. This change will not be disruptive for end users, since they can still use the current 8.x-1.0-alpha2 release for as long as they are on Drupal 8.6, and will automatically update to the upcoming alpha3 release when they move to Drupal 8.7. The release of 8.7 is planned for next week, and at that point 8.6 will be deprecated anyway. We could plan a release for OG shortly after Drupal 8.7 is released. |
Thanks for review! |
I agree, good call.
Sounds like a good plan. |
The first release candidate of Drupal 8.7 has been released. This is a good moment to start supporting Drupal 8.7, and start experimental support of Drupal 8.8.