← Back to issue list

yaml_utils: promote module to a package

View original Github issue

Metadata

Project
snapcraft
Number
#3385
Type
pull request
State
merged
Author
cjp256
Labels
Created
Updated
Closed

Current evaluation

Merged after rebasing to resolve conflicts. Promoted yaml_utils to a package by moving error handling and tests to snapacraft.yaml_utils.errors. Approved and passed CI.

Suggested action:

No scores available.

Issue body

Depends on #3383 The module is coupled to a project error that is really a yaml error. Move this into snapacraft.yaml_utils.errors and and correspondingly for its associated unit test.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after rebasing to resolve conflicts. Promoted yaml_utils to a package by moving error handling and tests to snapacraft.yaml_utils.errors. Approved and passed CI.
qwen3.6-35b-a3b-mtp-q6 Merged after rebasing to resolve conflicts. Promoted yaml_utils from a module to a package, relocating error handling to snapacraft.yaml_utils.errors and updating associated unit tests.
qwen3.6-35b-a3b-mtp-q6 Merged after resolving conflicts via rebase. Refactored the yaml_utils module into a package, relocating YAML error handling to snapacraft.yaml_utils.errors and updating corresponding unit tests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2781 yaml_utils: move _load_yaml from project for re-use (and add tests) snapcraft merged Merged. Moved _load_yaml to yaml_utils as load_yaml_file, added tests, and improved error handling to raise YamlValidationError. Approved, passed CI, and changed 3 files. Logic will be reused for snapcraft config files.
77%