Test against current core minor (10.1), previous (10.0) and next (10.2)

Results:

1. ❌ Failed on 10.0.x (which uses CKEditor 5 v35.4.0): https://git.drupalcode.org/project/entity_embed/-/jobs/222870
2. ✅ Passed on 10.1.x (which uses CKEditor 5 v38.0.1 in 10.0.0 and v38.1.0 in 10.1.0 → and v39.0.1 in 10.1.3 → : https://git.drupalcode.org/project/entity_embed/-/jobs/222854
3. ✅ Passed on 10.2.x (which uses CKEditor 5 v39, and soon v40 — see 📌 [upstream] Update CKEditor 5 to 40.0.0 Needs review ): https://git.drupalcode.org/project/entity_embed/-/jobs/222855

This that this actually would be a pretty strong reason to:

1. 2.0.x of Entity Embed for Drupal 10.0.x, but recompile for 10.0.x
2. 2.1.x for Drupal 10.1.x (straight branch from 8.x-1.x)
3. 8.x-1.x of Entity Embed for Drupal 9.5.x (revert ✨ Drupal 10 & CKEditor 5 readiness Fixed )
4. in the future, 2.2.x for Drupal 10.2.x or 10.3.x or …

@Dave Reid, I briefly mentioned the above last Friday, but I'm not sure you fully realized this. I suspect you won't like this 😅 But unfortunately it's out of my hands — it's a policy thing on the CKEditor 5 side.

They are working on doing fewer breaking changes — which is why they didn't go from https://github.com/ckeditor/ckeditor5/releases/tag/v38.0.0 directly to https://github.com/ckeditor/ckeditor5/releases/tag/v39.0.0, but did https://github.com/ckeditor/ckeditor5/releases/tag/v38.1.0 and tried doing https://github.com/ckeditor/ckeditor5/releases/tag/v38.2.0-alpha.1, but eventually never tagged a 38.2.0, and instead had to go to https://github.com/ckeditor/ckeditor5/releases/tag/v39.0.0.

But they have still got some ways on maturing into avoiding breaking changes.

I just wrote "2.0.x for Drupal 10.0.x", but I think it's equally reasonable for Entity Embed to require Drupal 10.1.x, and just say that that is the first and only version to support CKEditor 5.

P.S.: the self-RTBC here is only for the added test jobs. But still, it's failing on 10.0.x, so really … this need discussion with @Dave Reid to determine how he wants to proceed here.

Dave Reid → made their first commit to this issue’s fork.

I think I've got it covered by skipping the CKEditor5 test on 10.0, using the new GitLab CI variables for OPT_IN_TEST_PREVIOUS_MAJOR, OPT_IN_TEST_PREVIOUS_MINOR, and OPT_IN_TEST_NEXT_MINOR.

Everything is passing now. Merging.

👍 Looking good! 😊

Automatically closed - issue fixed for 2 weeks with no activity.