plainbox spread tests: use https for the git source entries
Metadata
Current evaluation
Merged a one-line change updating plainbox spread tests to use HTTPS for git source entries. Approved by two reviewers and passed CI checks.
Suggested action: —
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] 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 one-line change updating plainbox spread tests to use HTTPS for git source entries. Approved by two reviewers and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updated plainbox spread tests to use HTTPS for git source entries. The catkin legacy-pull test failed but did not block the merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updated plainbox spread tests to use HTTPS for git source entries. CI reported only a legacy catkin test failure, which did not block the merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2799 Plainbox test updates | snapcraft | merged | Merged plainbox test updates involving minor changes across three files. Approved by one reviewer with zero unresolved comments and no CI failures. | |
| #2863 spread tests: use source-depth: 1 for plainbox tests | snapcraft | merged | Merged after approval and CI validation. Updated spread tests to use source-depth: 1 for plainbox tests, modifying two files with three added lines. | |
| #2579 plainbox spread tests: lock down python packages | snapcraft | merged | Merged to lock down Python packages in plainbox spread tests, preventing dependency conflicts and ensuring consistent results. Approved by one reviewer, passed CI, modified five files, and increased coverage by 0.01%. | |
| #2321 tests: add spread suite for plainbox plugin (#2317) | snapcraft | merged | Merged after approval and passing CI. The pull request adds a Spread test suite for the plainbox plugin, pins the checkbox-ng package, and removes a misplaced test from the snapcraft test suite. | |
| #2317 tests: add spread suite for plainbox plugin | snapcraft | merged | Merged after approval and passing CI checks. Introduces a Spread test suite for plainbox plugin integration tests and pins the checkbox-ng package in the test environment. | |
| #1642 tests: move the plainbox test to the integration suite | snapcraft | merged | Merged changes to move the plainbox test to the integration suite. Updated expected namespace results to remove outdated year prefixes, fixing a test failure caused by upstream plainbox changes. Approved and passed CI. | |
| #2216 spread tests: keep sources local | snapcraft | merged | Merged PR #2216 to replace remote FTP sources in spread tests with in-tree tar archives, preventing test failures from server downtime. Approved by one reviewer, passed CI checks, and slightly increased code coverage. |