← Back to issue list

YAML loading either needs simplification or better testing

View original Github issue

Metadata

Project
craft-application
Number
#24
Type
issue
State
open
Author
lengau
Labels
Created
2023-06-08 21:22:38+00:00
Updated
2025-05-23 17:27:09+00:00
Closed

Current evaluation

No evaluation has been recorded for this issue yet.

Issue body

See [this comment thread](https://github.com/canonical/craft-application/pull/20/files/88c186c8a772c0ad8f5a8cfeb8fe84d352729111#diff-e1faa3a7dda894734ce22e2d235c8e3a1fd7e0ad2ae234fff07ed0fe131af6d3). The modifications made to the YAML loader may be more than is needed with the latest pyyaml, but I'm not confident enough to say so. If the questioned `except` clause is needed, we should have a test for it that has it raise that error, as right now the loader fails with the same issue before it gets to the dict constructor.

Evaluation history

No evaluation history available.