← Back to issue list

Removed the long time deprecated 'build' command (CRAFT-1109).

View original Github issue

Metadata

Project
charmcraft
Number
#798
Type
pull request
State
merged
Author
facundobatista
Labels
Created
Updated
Closed

Current evaluation

Merged removal of the deprecated 'build' command. Validator controls moved to Pack and booleans simplified. Changed 7 files (+92/-374). Approved and passed CI. Builder migration deferred to a follow-up PR.

Suggested action:

No scores available.

Issue body

The Validator, also used from `pack`, no longer makes sense: - bases_index: moved this control to the Pack command. - `from`: this was filled with the result of --project-dir, which already verified that the directory existed - `destructive_mode`, `debug`, `shell`, `shell_after` and `force`: just converted an already bool to bool Left for next PR: move the `Builder` class and helper function/constants to the `pack.py` file (which would be a huge diff, specially because of tests, but without semantic changes).

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged removal of the deprecated 'build' command. Validator controls moved to Pack and booleans simplified. Changed 7 files (+92/-374). Approved and passed CI. Builder migration deferred to a follow-up PR.
qwen3.6-35b-a3b-mtp-q6 Merged removal of the deprecated build command. Validator logic was refactored, moving bases_index control to the Pack command and removing unused fields. Bool conversions simplified. Builder class migration deferred to a subsequent PR.
qwen3.6-35b-a3b-mtp-q6 Merged removal of the deprecated build command. Validator logic was refactored, moving bases_index control to the Pack command and simplifying boolean flags. Builder class migration to pack.py is deferred.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#797 Removed the --requirements option for build and pack commands (CRAFT-787). charmcraft merged Merged after removing the --requirements option from build and pack commands, resolving issue #657. Approved by one reviewer. Changes span six files, deleting 315 lines and adding 16.
70%
#557 Removed the command group from the command itself to simplify the whole structure (CRAFT-555). charmcraft merged Merged following reviewer approval. Simplified the architecture by removing the command group from the command itself (CRAFT-555). The update modified 12 files (+165/-173 lines), primarily adjusting tests while flagging core structural changes.
70%
#739 Hide the 'build' command and make 'pack' the common one (CRAFT-938). charmcraft merged Merged after approval, this change hides the 'build' command and promotes 'pack' as the common command, resolving #720. The update modifies four files with four additions and three deletions.
70%
#683 Issue a deprecation notice when using the 'build' command (CRAFT-768). charmcraft merged Merged implementation of a deprecation notice for the 'build' command (CRAFT-768). Approved by one reviewer, the update modifies three files with 21 additions and zero deletions.
70%