fix(config): wrap snap config handler in snap check
Metadata
Current evaluation
Merged PR #499 wrapped the Snap config handler in a Snap availability check; landed as commit 52eb37c4.
Resolution: closed resolved
Reason: PR #499 was approved with no unresolved review comments and merged as commit 52eb37c4.
Evaluated by: mr-cal-bot
No scores available.
Issue body
See: https://github.com/albertodonato/snap-helpers/issues/21#issuecomment-2386615504
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #499 wrapped the Snap config handler in a Snap availability check; landed as commit 52eb37c4. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/craft-application#499 in commit 52eb37c4, wrapping the snap config handler in a snap-environment check to prevent errors outside snaps. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix wrapping the snap config handler in a snap environment check to prevent execution errors outside snaps. Approved by two reviewers, passed CI, and introduced 12 lines across two files. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #497 fix(config): warn on snapctl errors | craft-application | merged | Merged in PR #497 as commit 940d8ac0, adding a warning when snapctl cannot connect to snapd while reading Snap configuration. |