← Back to issue list

cli: remove enable-ci command

View original Github issue

Metadata

Project
snapcraft
Number
#3164
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #3164 (commit ccb612e5a), removing enable-ci after export-login superseded it.

Resolution: closed resolved

Reason: PR #3164 was merged as commit ccb612e5a, deleting the obsolete command and its 753 lines of implementation. The maintainer documented export-login, added by PR #1780, as its replacement.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The command has been superseeded by use of export-login. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #3164 (commit ccb612e5a), removing enable-ci after export-login superseded it.
stealth/space-bunny-alpha — closed resolved Merged PR #3164 (commit ccb612e5a) deleted the legacy enable-ci command (-753 lines, 7 files), which had been superseded by `export-login`; no enable-ci references remain in the tree.
stealth/space-bunny-alpha — closed resolved Merged PR #3164 (commit ccb612e5a) removed the enable-ci command from the snapcraft CLI, deleting 753 lines across 7 files, because it was superseded by the export-login command.
stealth/space-bunny-alpha — closed superseded Merged PR #3164 (commit ccb612e5a) deleted the `enable-ci` command (-753 lines across 7 files) because `export-login` superseded it; approved with no CI failures.
stealth/space-bunny-alpha — closed resolved Merged PR #3164 (commit ccb612e5a) removed the enable-ci command because export-login superseded it.
qwen/qwen3.6-35b-a3b — — Merged removal of the enable-ci CLI command, superseded by export-login. Approved by one reviewer, passed CI checks, and removed 753 lines across seven files.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of the enable-ci CLI command, superseded by export-login. Authored by sergiusens and integrated into the main branch.

Update history

No update history recorded yet.

Related work

  • Superseded By: snapcraft#1780 (confidence 99%)

    Added export-login, the credential-export workflow cited as replacing enable-ci.

Related issues

Issue Project State Summary Similarity
#1795002 Remove enable-ci command. snapcraft (launchpad) closed Removed enable-ci in snapcraft#3164 (commit ccb612e5a); CI authentication is instead handled by exporting login credentials with export-login.
79%
#1780 cli: add export-login command snapcraft merged Merged as commit 53027208d ("cli: add export-login command (#1780)"), adding snapcraft export-login plus a --with option to login, closing #1711 and the abandoned enable-ci approach.
74%
#1711 Develop enable-ci circle-ci snapcraft closed CircleCI enable-ci was replaced by export-login with documentation; PR #1780 added the command and merged as commit 53027208d, satisfying #1711.
73%