- π«π·France andypost
As related showing it still good to have #3205480-18: Drop PECL YAML library support in favor of only Symfony's YAML? β
- Merge request !5641Add hook_requirements() for PECL YAML parser #2157447 β (Closed) created by andypost
- Status changed to Needs review
9 months ago 2:48pm 1 December 2023 - Status changed to Needs work
9 months ago 3:26pm 1 December 2023 - π¬π§United Kingdom alexpott πͺπΊπ
I don't know if we have to recommend the PECL yaml parser. I think we could indicate what's be used for YAML parsing on the status report and link to a page on drupal.org that discusses the possibilities..
- Status changed to Needs review
9 months ago 9:21pm 5 December 2023 - π«π·France andypost
The MR already provides informing
Your server is not using the PECL YAML extension for parsing YAML. Installing this extension will improve the performance of parsing YAML files on your server.
maybe it needs section at https://www.drupal.org/docs/getting-started/system-requirements/php-requ... β with the link from report/status
- π§πͺBelgium Wim Leers Ghent π§πͺπͺπΊ
"will improve" sounds like a recommendation to me? π
Left a suggestion for changing it to "merely informing", i.e. purely observing.
- Status changed to Needs work
9 months ago 4:41pm 11 December 2023 - πΊπΈUnited States smustgrave
Agree that
Installing this extension will improve the performance of parsing YAML files on your server.
sounds like a recommendation, or. that not doing it means I'm doing something wrong.
But maybe that sentence could be added to a section in https://www.drupal.org/docs/getting-started/system-requirements/php-requ... β that @andypost recommended.
So maybe
Your server is not using the PECL YAML extension for parsing YAML. See @link for more information.
- π«π·France andypost
It becoming a follow-up for π PECL YAML library supports YAML 1.1, but Symfony supports YAML 1.2 Active
- Status changed to Closed: outdated
8 months ago 11:57am 22 January 2024