← Back to issue list

added test for git sources in the get() method in snapcraft/sources.py

View original Github issue

Metadata

Project
snapcraft
Number
#217
Type
pull request
State
closed
Author
fazerlicourice7
Labels
—
Created
Updated
Closed

Current evaluation

Contributor submitted a test for git sources, but the PR closed without merging. Reviewers requested copyright updates, PEP8 fixes, and commit squashing. The author struggled with git rebase and abandoned the PR to start over.

Suggested action: —

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Contributor submitted a test for git sources, but the PR closed without merging. Reviewers requested copyright updates, PEP8 fixes, and commit squashing. The author struggled with git rebase and abandoned the PR to start over.
qwen3.6-35b-a3b-mtp-q6 — — Adding a test for git sources was closed without merging. Following review feedback and CI failures, the contributor struggled with git squashing and abandoned the work to recreate it.
qwen3.6-35b-a3b-mtp-q6 — — Closed without merging. After review requested PEP8 fixes, copyright updates, and commit squashing, the contributor struggled with git rebase and abandoned the pull request to resubmit.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#216 Added test for sources.py snapcraft closed Added a test for Git sources in _get_source_type_from_uri. The request closed without merging. Zero reviews and no CI checks indicate the change was abandoned due to inactivity.
76%
#1534411 Needs Test for Git sources snapcraft (launchpad) closed Closed without action. The request to add unit tests for Git sources in snapcraft/sources.py was abandoned, with zero comments and no implementation.
76%
#233 added test for git sources in sources.py snapcraft merged Merged after adding tests for git sources in sources.py. Initial assertion fix request was dropped after clarifying the method needs no parameters. Coverage improved, approval granted, and changes merged.
73%