← 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 #3385 as commit 23059ae25, promoting yaml_utils to a package and moving YAML-specific error handling and its unit tests into snapcraft.yaml_utils.errors.

Resolution: closed resolved

Reason: PR #3385 was merged into Snapcraft as commit 23059ae25. The commit restructured yaml_utils and moved its project-specific error into the new snapcraft.yaml_utils.errors module.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged #3385 as commit 23059ae25, promoting yaml_utils to a package and moving YAML-specific error handling and its unit tests into snapcraft.yaml_utils.errors.
stealth/space-bunny-alpha — closed resolved Merged PR #3385 as commit 23059ae25, promoting yaml_utils to a package and moving YAML error handling and its unit tests to snapcraft.yaml_utils.errors after rebasing.
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.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#3383 (confidence 100%)

    Stated dependency; merged as commit d2f318468 before #3385.

Related issues

Issue Project State Summary Similarity
#2781 yaml_utils: move _load_yaml from project for re-use (and add tests) snapcraft merged Merged in #2781 (commit 89bc9695c): moved YAML loading into yaml_utils as load_yaml_file, added tests in f1e9d4acf, and normalized YAML errors to YamlValidationError.
71%