← Back to issue list

tests: use python2 to check the CLA

View original Github issue

Metadata

Project
snapcraft
Number
#1044
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

PR #1044 merged as commit 5d36cf3f ('ci: use python2 to check the CLA'), switching the Travis CLA check to Python 2 for LP bug #1655790; Travis passed and @kyrofa approved.

Resolution: closed resolved

Reason: The PR itself was merged into master as commit 5d36cf3f, changing the CLA check script to invoke python2 (a 1-file, +5/-1 change) so the contributor licence check stopped failing on Travis. @kyrofa confirmed 'Travis passed' and the change landed.

Evaluated by: mr-cal-bot

No scores available.

Issue body

LP: [#1655790](https://bugs.launchpad.net/snapcraft/+bug/1655790)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #1044 merged as commit 5d36cf3f ('ci: use python2 to check the CLA'), switching the Travis CLA check to Python 2 for LP bug #1655790; Travis passed and @kyrofa approved.
stealth/space-bunny-alpha — closed resolved Merged PR #1044 in commit 5d36cf3f, changing CLA checks to use Python 2; Travis passed and @kyrofa approved.
qwen/qwen3.6-35b-a3b — — Merged changes to update tests for Python 2 CLA verification. The pull request modified one file, passed Travis CI, maintained 100% diff coverage, and was approved by kyrofa.
qwen3.6-35b-a3b-mtp-q6 — — Merged a test configuration update to use Python 2 for CLA verification, resolving Launchpad bug #1655790. The change passed Travis CI and Codecov checks without altering overall coverage.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#1106 (confidence 45%)

    Follow-up commit 56e934447 'tests: check the CLA on all the commits' further extended the CLA check

  • Related To: snapcraft#3368 (confidence 40%)

    Commit ead57405c 'ci: use github actions for cla check' later replaced the Travis-based CLA check that this PR touched

Related issues

Issue Project State Summary Similarity
#1043 tests: fix the CLA launchpadlib install in travis snapcraft closed PR #1043 was merged into master, fixing the Travis CLA check's launchpadlib installation; the PR head is commit f673cc3d93fe368f8f99ce6387f0872c2ad832aa.
75%
#1106 tests: check the CLA on all the commits snapcraft merged Merged PR #1106 updated tests to check the CLA across every commit in a branch; merged as commit 56e934447.
73%