Exceptions from errors in services.yml files should tell you which file

Ran the test locally for #44 and it fails with

Failed asserting that exception message 'Unable to parse at line 1 (near "   do not:").' contains 'The file "vfs://drupal/modules/example/example.yml" does not contain valid YAML'.

Tried breaking an existing service but could not but the test shows the issue just fine I believe.

Looks good!

Is this a case where we should be using assertions instead of exceptions? See for example #2454649: Cache Optimization and hardening -- [PP-1] Use assert() instead of exceptions in Cache::merge(Tags|Contexts) → . Since there are other similar exceptions in the same file, maybe a follow-up?

Opened 📌 YamlFileLoader use assertions instead of exceptions Closed: works as designed

Committed e53445e and pushed to 10.1.x. Thanks!

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