← Back to issue list

ci: use travis conditionals

View original Github issue

Metadata

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

Current evaluation

PR #1560 merged as commit caa20e0f5, refactoring .travis.yml to gate jobs with Travis conditionals (+20/-10).

Resolution: closed resolved

Reason: The PR was approved and merged into snapcraft as commit caa20e0f5 ("ci: use travis conditionals"), which rewrote the .travis.yml build matrix to use native Travis conditional stages instead of manual job ordering.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #1560 merged as commit caa20e0f5, refactoring .travis.yml to gate jobs with Travis conditionals (+20/-10).
stealth/space-bunny-alpha — closed resolved Merged PR #1560 (commit caa20e0f5) by maintainer come-maiz, replacing shell-based conditionals in .travis.yml with native Travis `if:` conditionals (+20/-10 lines) to gate CI job steps. Later refined by #2523 and Travis itself removed in #5073.
stealth/space-bunny-alpha — closed resolved Merged PR #1560 (commit caa20e0f5) by come-maiz, refactoring .travis.yml to use Travis conditionals for CI job control (+20/-10 lines).
stealth/space-bunny-alpha — closed resolved Merged in canonical/snapcraft#1560 (commit caa20e0f5), updating .travis.yml to use Travis conditionals (+20/-10 lines).
qwen/qwen3.6-35b-a3b — — Merged pull request updating CI configuration to use Travis conditionals. The change modified one file (+20/-10 lines) and received approval from one reviewer before being integrated.
qwen3.6-35b-a3b-mtp-q6 — — Merged a pull request updating the CI configuration to use Travis conditionals. All contributing guidelines and test suites passed prior to integration.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#2523 (confidence 60%)

    Later commit 4900f7323 "ci: improve travis integration conditionals" further refined the same conditional structure introduced here.

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

    Commit 5e91afb6a "chore: remove travis and autopkgtest" eventually deleted .travis.yml, making this CI config obsolete.

Related issues

Issue Project State Summary Similarity
#2523 ci: improve travis integration conditionals snapcraft merged Merged as commit 4900f7323: .travis.yml spread/integration test conditionals fixed, unused cron trigger removed, integration script indentation normalized.
81%