PHP version compatibility

My merge requests adds back support for PHP 7.4

It looks like the issue was caused by the trailing comma on this line:
https://git.drupalcode.org/project/environment_indicator/-/merge_request...
This syntax is valid in newer PHP versions, but breaks on PHP 7.4, which the module currently supports.

To help catch similar issues, I’ve added a PHPUnit test to lint PHP files across supported versions:
https://git.drupalcode.org/project/environment_indicator/-/merge_request...

While this won’t catch the trailing comma specifically (as PHP 7.4 is no longer available in the GitLab runner environment), it sets us up for stricter compatibility checks going forward. I’d recommend holding off on using syntax that requires PHP 8.0+ until the module drops support for older versions in a future minor release.

I am also adding this issue to the 🌱 [META] Release Plan for Environment Indicator Patch Release 4.0.21 Needs review

If changes like this are going to be made, it makes it very urgent to get a branch which is aimed at currently-supported versions of Drupal. This change introduces a coding standards error. Contributing to this module will become more difficult because people will need to remember to write in an old version of PHP. remembering what changes have been made and what coding standard issues to ignore.

I agree that maintaining compatibility with PHP 7.4 makes contributions trickier, and there’s value in targeting modern environments. That said, this patch is specifically scoped to fix a regression introduced in a patch release (4.0.18–4.0.21) that broke sites still running PHP 7.4 — without any documented change in requirements.

To avoid repeating that mistake, I’d suggest the following sequence:

1. Merge this fix to restore compatibility in 4.x and ensure sites on PHP 7.4 can safely upgrade within the same major version.
2. Create a final 4.0.x release that:
   • Maintains support for Drupal 9.3 through 11.
   • Avoids changes that break backward compatibility for supported environments.
3. After that, address the syntax issue in 🌱 Drop support for versions of Drupal older than 10.3 Active , which also drops support for Drupal core versions earlier than 10.3.
4. Then create a 4.1.0-beta1 minor release

We’d appreciate review on any of the “Needs review” issues blocking the final 4.0.x release:
🌱 [META] Release Plan for Environment Indicator Patch Release 4.0.21 Needs review or 🌱 Draft: Release Plan for Environment Indicator 4.1.0-beta1 Active

I think it would be reasonable to fix just the regression and then move on to 4.1.x.

I've updated the PHPUnit tests so that they pass in all of the gitlab-ci jobs.

While updating the PHPUnit Test I was testing the module using the combination of PHP 8.3 and Drupal Core 11.0.13 as well.

Also, it appears that all of the gitlab-ci tests are currently passing due to this issue: 💬 Validate stage may use outdated artifact, leading to flaky test results Active