- Issue created by @mably
- Merge request !28Issue #3476690 by mably: cspell, estlint, stylelint, phpcs and phpstan fixes → (Open) created by mably
- 🇨🇦Canada _randy
I should have pushed my D11 info.yml compatibility a while ago -- apologies for that. I pushed those simple yml file changes for now.
The linted output makes it difficult for me to determine what we're pushing into the repository.
I'll have to dedicate more time to look at the linted output as I see some changes to method/function declarations for nullable types. Perhaps just better shorthand for PHP 7.1+ -- just like some of the replaced is_set(..) with ?? operator.
Or we close this ticket down for d11 compatibility and open a new ticket where we can work through linting and code standard updates sequentially.
- 🇫🇷France mably
Going through all the phpcs and phpstan warnings... 😅
May be we should create a new 4.x dev branch for Drupal 10+ and PHP 8+?
- 🇫🇷France mably
All green now, the unit tests are finally passing after a huge work on the schema file. Yay! 🎉
- Status changed to Needs review
22 days ago 11:08am 26 November 2024 - 🇫🇷France mably
Sadly, quite a few things are broken now, will see what I can do.