← Back to issue list

travis: set cla check as final stage

View original Github issue

Metadata

Project
snapcraft
Number
#3233
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #3233 (commit 2eeafcb9b, 'ci: set cla check as final stage') reordered .travis.yml so the CLA check runs last, keeping test results visible for contributors with unresolved CLAs.

Resolution: closed resolved

Reason: The PR was approved and merged into master as commit 2eeafcb9b, which appears in the .travis.yml history. It moved the CLA stage to the end of the Travis stage list so test output is still reported regardless of CLA status.

Evaluated by: mr-cal-bot

No scores available.

Issue body

It's useful to see test results regardless of the outcome of the CLA checks. This may encourage further development of PRs while contributors work on getting their CLAs sorted out. Signed-off-by: Chris Patterson <chris.patterson@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
stealth/space-bunny-alpha — closed resolved Merged PR #3233 (commit 2eeafcb9b, 'ci: set cla check as final stage') reordered .travis.yml so the CLA check runs last, keeping test results visible for contributors with unresolved CLAs.
stealth/space-bunny-alpha — closed resolved Merged #3233 in commit 2eeafcb9b, making the Travis CLA check the final stage so test results remain visible while contributors address CLA status.
stealth/space-bunny-alpha — closed resolved Merged in #3233 via commit 2eeafcb9b, making the Travis CLA check the final stage so test results remain visible while contributors resolve CLA issues.
qwen/qwen3.6-35b-a3b — — Merged a Travis CI configuration update that moves the CLA check to the final stage. This ensures test results display regardless of CLA status, enabling ongoing development while contributors resolve compliance. Approved and passed all checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged a Travis CI update moving the CLA check to the final stage. Test results now display regardless of CLA status, encouraging contributors to continue PR development while resolving licensing requirements.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#3368 (confidence 70%)

    Later migrated the CLA check from Travis to GitHub Actions, superseding this Travis-stage ordering change.

  • Related To: canonical/snapcraft#5073 (confidence 65%)

    Travis configuration was eventually removed entirely, retiring this CLA-stage arrangement.

Related issues

Issue Project State Summary Similarity
#1482 ci: skip the CLA check for pull requests from the bot snapcraft merged Merged maintainer PR #1482 (commit 305b46d34) — a 2-line .travis.yml change skipping the CLA check on pull requests opened by the bot; approved by @sergiusens.
72%
#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.
71%
#2477 ci: shallow clones for CLA checks on travis snapcraft merged Merged #2477 via commit 7060eb45f, adding shallow clones for Travis CLA checks to reduce CI execution time.
70%