Problem/Motivation

Drupal 10 and Drupal 11 have different versions of phpunit.
In Drupal 11 with PhpUnit 11, we can use attributes. In PhpUnit 10 we cannot.

In the end this module will run on a Drupal 11 or 12 website.

It made sense for a while to test with Drupal 10, just to see if some old stuff was working in 10 but not 11, given that the module was initially ported to an even earlier core version (8 or 9).

But now we can drop it I think.

localize also upgraded to Drupal 11 in https://gitlab.com/drupal-infrastructure/sites/localize/-/commit/992fcb7....

Steps to reproduce

Proposed resolution

Stop all CI checks with Drupal 10.

UPDATE: Actually just fully drop Drupal 10 support.
(that means update the *.info.yml files, because our composer.json says nothing about drupal core)

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork l10n_server-3564210

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

donquixote created an issue. See original summary.

donquixote opened merge request !122

donquixote’s picture

donquixote commented
Title: Stop testing with Drupal 10 » Drop Drupal 10

Let's just go all the way and remove older Drupal versions from *.info.yml files.

I am not sure where else we have to change anything.
At least we now know that we can refactor everything with Drupal 11 in mind, or even 11.3.

donquixote’s picture

donquixote commented
Status: Active » Needs review
donquixote’s picture

donquixote commented

If people still want to test with Drupal 10 locally, they can just revert the commit.

donquixote’s picture

donquixote commented

Actually... localize project is still on Drupal 10!
Do we care?

EDIT: No it is not :)

donquixote’s picture

donquixote commented
Issue summary: View changes
donquixote’s picture

donquixote commented
Priority: Normal » Major

It is blocking the CI now for other issues..

pierregermain’s picture

pierregermain commented
Status: Needs review » Reviewed & tested by the community

The changes look good and accomplish the goal of dropping Drupal 10 support and unblocks the CI.

  • fmb committed b6dbd357 on 3.0.x 
    Merge branch '3564210-drop-drupal-10' into '3.0.x'

Resolve #3564210 "...

  • donquixote committed a923d01c on 3.0.x 
    Issue #3564210: Drop Drupal 10 and lower in *.info.yml files.
fmb’s picture

fmb commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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