Problem/Motivation
The phpstan CI job fails on the 10.1.x and 9.2.x branches: the updated phpstan-drupal analyzers raise drupal.symfonyYamlParse on pre-existing code in modules/vbp_text_and_image/includes/updates/v8.inc.
Steps to reproduce
Run the phpstan job on the branch pipeline.
Proposed resolution
- Use
\Drupal\Component\Serialization\Yaml::decode()instead of the Symfony YAML parser, which handles exceptions consistently and applies the correct parse flags.
Remaining tasks
- ✅ File an issue
- ❌ Addition/Change/Update/Fix
- ❌ Testing to ensure no regression
- ➖ Automated unit testing coverage
- ➖ Automated functional testing coverage
- ➖ UX/UI designer responsibilities
- ❌ Readability
- ➖ Accessibility
- ➖ Performance
- ➖ Security
- ➖ Developer Documentation
- ➖ User Guide Documentation
- ➖ Reviewed by human
- ➖ Code review by maintainers
- ➖ Full testing and approval
- ➖ Credit contributors
- ➖ Review with the product owner
- ➖ Release notes snippet
- ✅ Release
User interface changes
- N/A
API changes
- N/A
Data model changes
- N/A
Release notes snippet
- Modernized the YAML handling in the update helpers to the Drupal serialization component.
AI-Generated: Yes
Issue fork varbase_bootstrap_paragraphs-3615795
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:
Comments
Comment #6
rajab natshah✅ Released varbase_bootstrap_paragraphs-9.2.2
Comment #7
rajab natshahComment #9
rajab natshah✅ Released varbase_bootstrap_paragraphs-10.1.2
Comment #10
rajab natshahComment #11
rajab natshah