← Back to issue list

fix!(Application): remove run_managed

View original Github issue

Metadata

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

Current evaluation

Closed unmerged; the Application.run_managed cleanup was later merged in PR #1077 via commit e00540d1.

Resolution: closed superseded

Reason: PR #718 was approved but closed without merging and had failing checks. The same cleanup was tracked by issue #1062 and merged through PR #1077 as commit e00540d1, leaving #718 redundant.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This was supposed to be deleted for 5.0.0 - [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`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Closed unmerged; the Application.run_managed cleanup was later merged in PR #1077 via commit e00540d1.
stealth/space-bunny-alpha — closed stale Proposed removal of the obsolete Application.run_managed API for 5.0.0; the PR was approved but closed unmerged, with several lint and test checks failing.
qwen/qwen3.6-35b-a3b — — Merged to remove the deprecated run_managed method from the Application module, as planned for version 5.0.0. Approved by one reviewer. CI passed core checks despite minor lint and test failures.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/craft-application#1077 (confidence 98%)

    Merged as e00540d1, removing the managed-command APIs and directing managed runs through ProviderService.

  • Related To: canonical/craft-application#1062 (confidence 97%)

    Later issue explicitly tracked removal of Application.run_managed and was resolved by PR #1077.

Related issues

Issue Project State Summary Similarity
#1075 chore!: remove Application.run_managed craft-application closed Closed unmerged and replaced by #1077, which removed managed-command APIs and completed the ProviderService migration in commit e00540d1.
70%