← Back to issue list

sources: refactor tar source into module.

View original Github issue

Metadata

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

Current evaluation

Merged a refactor extracting the Tar source into a dedicated module at sources/_tar.py. The change adds two files, increases code coverage by less than 0.01%, and advances work on Launchpad bug #1648906.

Suggested action:

No scores available.

Issue body

This PR makes more progress on LP: [#1648906](https://bugs.launchpad.net/snapcraft/+bug/1648906) by extracting the Tar source into `sources/_tar.py`.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a refactor extracting the Tar source into a dedicated module at sources/_tar.py. The change adds two files, increases code coverage by less than 0.01%, and advances work on Launchpad bug #1648906.
qwen3.6-35b-a3b-mtp-q6 Merged into master, extracting tar source logic into a dedicated sources/_tar.py module. The refactor improves code organization and slightly increases test coverage while advancing ongoing bug resolution.
qwen3.6-35b-a3b-mtp-q6 Merged a refactor extracting Tar source handling into a dedicated sources/_tar.py module. The change improves code organization and increases overall test coverage by less than 0.01%.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#957 sources: refactor base sources into module. snapcraft merged Merged following two approvals and passing CI checks. Refactored Base and FileBase sources into sources/_base.py, increasing test coverage by 1.20% and progressing work on Launchpad bug #1648906.
74%
#961 sources: refactor local source into module. snapcraft merged Merged after approval and CI. Refactored the local source into sources/_local.py, advancing Launchpad bug #1648906. Modified four files with a +236/-191 line change and slightly increased overall code coverage.
72%
#45 sources: add the tar source handler craft-parts merged Added a tar source handler to enable tar files as a part source. Merged after approval by two reviewers, introducing 403 additions and 1 deletion across 10 files.
71%