← Back to issue list

Upload: Clean up output.

View original Github issue

Metadata

Project
snapcraft
Number
#283
Type
pull request
State
merged
Author
kyrofa
Labels
Created
Updated
Closed

Current evaluation

Merged improvements to snapcraft upload output, adding progress indicators and clarifying the OTP prompt to resolve Launchpad bug #1538692. Approved by reviewers. Test logger cleanup was deferred.

Suggested action:

No scores available.

Issue body

This PR fixes LP: [#1538692](https://bugs.launchpad.net/snapcraft/+bug/1538692) by change the OTP prompt to be more clear, and adding progress indicators. The upload output now looks something like this: ``` $ snapcraft upload Uploading foo_1.7_amd64.snap [==============================================================] 100% Checking package status... | # <-- this is an animated pinwheel indicator Application uploaded successfully (as revision 6) Please check out the application at: https://myapps.developer.ubuntu.com/dev/click-apps/1337/ ``` Note that the test logger output is not cleaned here-- I bisected it back to 23290980fe9dbbc407cb0edb3cd1a22f781d411b but I can't figure out what's causing it.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged improvements to snapcraft upload output, adding progress indicators and clarifying the OTP prompt to resolve Launchpad bug #1538692. Approved by reviewers. Test logger cleanup was deferred.
qwen3.6-35b-a3b-mtp-q6 Merged improvements to upload output, fixing Launchpad bug #1538692 with clearer OTP prompts and progress indicators. Approved by reviewers and integrated. Test logger cleanup deferred.
qwen3.6-35b-a3b-mtp-q6 Merged updates that enhance snapcraft upload output with progress indicators and clearer OTP prompts. Approved and merged following maintainer reviews. Test logger cleanup was deferred.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2469 cli: clean up snapcraft push output snapcraft merged Merged cleanup of the snapcraft push CLI output. Verbose messages were moved to debug level, filename formatting was standardized, and log coloring was adjusted. Approved by one reviewer, passed CI, and integrated into the main branch.
74%
#639 Use a snap that doesn't require a manual review for the upload test snapcraft merged Merged a change replacing a snap requiring manual review with an auto-approved alternative for the upload test, resolving Launchpad bug #1600078. The update modifies two files to streamline the test workflow.
73%
#277 Add documentation for how to use snapcraft upload. snapcraft merged Merged documentation detailing the snapcraft upload workflow. Resolves Launchpad bug #1539234. Approved by reviewers following minor file extension naming corrections.
71%