← Back to issue list

fix: improve the handling of platforms errors.

View original Github issue

Metadata

Project
craft-application
Number
#706
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #706 as commit b61fcd58, improving platform-error handling and fixing internal errors for build-on configurations in #690.

Resolution: closed resolved

Reason: PR #706 was approved and merged with all CI checks passing. Commit b61fcd58 delivers the platform-error fixes linked to CRAFT-4369 and CRAFT-4370.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Fixes #690 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? - [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- CRAFT-4369, CRAFT-4370

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #706 as commit b61fcd58, improving platform-error handling and fixing internal errors for build-on configurations in #690.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-application#706 to improve platform-error handling, fixing #690 (CRAFT-4369 and CRAFT-4370); all CI checks passed.
qwen/qwen3.6-35b-a3b — — Merged fix for platform error handling. Approved by two reviewers with all CI checks passing. Resolves issue #690 and CRAFT-4369/4370. Modifies eight files with 98 additions and 239 deletions.

Update history

No update history recorded yet.

Related work

  • Related To: craft-application#690 (confidence 100%)

    The PR explicitly states that it fixes the internal platform errors reported in #690.

Related issues

Issue Project State Summary Similarity
#686 fix(ProjectService): better handling for invalid platforms craft-application merged Merged PR #686 as commit 651f7573, improving ProjectService handling of invalid platforms and updating the changelog; changes originated from review feedback on #683.
74%
#687 fix(commands/lifecycle): Pass the correct platform and build-for craft-application merged Merged PR #687 as commit 5cf3d407, fixing lifecycle commands to pass the correct platform and build-for values to managed instances.
71%