← Back to issue list

ci: don't publish snap on push to master

View original Github issue

Metadata

Project
snapcraft
Number
#3449
Type
pull request
State
merged
Author
cjp256
Labels
Created
Updated
Closed

Current evaluation

Merged a CI update removing automatic snap publishing on master pushes. The change deletes four lines of CI code as publishing is handled externally for all architectures. Approved and merged cleanly.

Suggested action:

No scores available.

Issue body

This process happens externally (for all architectures). - [ ] 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
qwen/qwen3.6-35b-a3b Merged a CI update removing automatic snap publishing on master pushes. The change deletes four lines of CI code as publishing is handled externally for all architectures. Approved and merged cleanly.
qwen3.6-35b-a3b-mtp-q6 Merged to disable CI snap publishing on master pushes, as the process is now handled externally for all architectures.
qwen3.6-35b-a3b-mtp-q6 Merged updates to the CI configuration that disable automatic snap publishing on master branch pushes, as the process is handled externally for all architectures.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2900 ci: publish the CI built snap to the Snap Store snapcraft merged Merged into master. Adds CI configuration to automatically publish built snaps to the Snap Store. Approved by a maintainer, passed all checks, and introduced 29 lines across two files with 100% diff coverage.
79%
#352 ci(snap): publish snap for PRs and feature branches rockcraft merged Merged CI workflow update to publish Snap packages for pull requests and feature branches. Approved by two reviewers, changing one file with 25 additions and one deletion.
77%
#2899 ci: publish the CI built snap to the Snap Store snapcraft closed CI snap publishing changes were closed without merging. Lacking reviewer feedback and with two unresolved comments, the pull request was abandoned. No CI checks executed.
72%
#2045 ci: separate snap build and PR publish jobs charmcraft merged Merged CI changes separating snap build and PR publish jobs. Approved by two reviewers, this update allows concurrent snap testing while the store processes branch publishing, streamlining the release workflow.
72%
#3987 ci: remove docker before building snap snapcraft merged Merged into main after reviewer approval and passing CI checks. The change modifies four files to remove Docker before building the snap, with no impact on overall code coverage.
71%
#5248 ci: publish snaps to arm64 snapcraft merged Merged CI updates to build and publish test snaps for arm64. Approved by two reviewers with passing checks, the change enables automated testing for related snapcraft and craft-providers issues.
71%
#815 ci: fix publishing workflow craft-application merged Merged a single-line fix to the CI publishing workflow. Approved by two reviewers with all continuous integration checks passing.
70%