← Back to issue list

refactor: migrate to starflow

View original Github issue

Metadata

Project
snapcraft
Number
#5225
Type
pull request
State
merged
Author
bepri
Labels
Created
Updated
Closed

Current evaluation

Merged a large refactor migrating the codebase to starflow across 173 files. CI failures were dismissed as pre-existing flaky tests, unrelated security checks, and legacy commit formatting. Rebased on main before merge.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox run -m lint`? - [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`) ----- Should be reviewable on a commit-by-commit basis. Comments will be left on individual files explaining why some changes were necessary. As some words of encouragement: about 250 of the file changes were purely from `make format`!

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a large refactor migrating the codebase to starflow across 173 files. CI failures were dismissed as pre-existing flaky tests, unrelated security checks, and legacy commit formatting. Rebased on main before merge.
qwen3.6-35b-a3b-mtp-q6 Migrated to starflow with formatting and docs updates. Merged despite failing tests caused by pre-existing flakiness and unrelated validation checks. Starbase history integration deferred.
qwen3.6-35b-a3b-mtp-q6 Merged the starflow migration refactor post-rebase. Proceeded despite failing tests due to pre-existing flaky spread tests, an unrelated osv scanner issue, and a conventional commits check conflict with starbase history. Docs updated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#71 fix(ci): use main branch from starflow imagecraft merged Merged a fix updating the CI configuration to use the main branch from starflow. The change modified two files with two additions and two deletions.
76%
#622 build: switch to starflow craft-application merged Merged. The build system was switched to starflow after two approvals. Mandatory CI checks were updated to specific lint and test jobs, the branch was rebased, and the changes were merged.
76%
#72 fix(ci): use main branch from starflow imagecraft merged Merged a maintainer's CI fix to use the main branch from starflow. The change updated 2 files with a +2/-2 line diff.
76%
#143 chore: migrate to starflow and Makefile craft-archives merged Migrated to starflow and a new Makefile. Linting and uv.lock modification issues were fixed during review, alongside switching formatting from yamllint to prettier. Approved by two reviewers and merged after CI passed.
76%
#2015 build: use starflow charmcraft merged Merged the build system update to use starflow. Approved by two reviewers, the change modifies 22 files. Final review noted pending spread test fixes, but the PR was successfully merged.
75%
#241 chore: move tests to starflow craft-store merged Merged after two approvals and passing CI. Tests were relocated to starflow across 43 files, featuring a major autoformat pass. Minor lock file discussion preceded integration.
74%
#260 ci: use starflow renovate config starbase merged Merged the starflow renovate configuration update, removing 220 lines of CI setup across two files. Approved by two reviewers and successfully integrated.
74%
#2027 build(ci): use starflow for policy and security scans charmcraft merged Merged update to the CI pipeline using starflow for policy and security scans. Approved by two reviewers, rebased on main, and reduced code by 12 lines across four files.
74%
#782 Work/craft 3831 starflow rockcraft merged Merged by maintainer tigarmo following approval from two reviewers. The changeset adds 3950 lines and removes 1433 across 48 files, implementing the Starflow workflow updates.
73%
#5202 ci: replace Renovate config with starflow snapcraft merged Merged following two approvals and passing CI checks. Replaced the local Renovate configuration with the shared starflow config, removing 157 lines.
73%