← 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

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.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Replaces CI configuration with Travis conditionals to optimize build matrix execution. Integrated directly by maintainer come-maiz without additional review comments.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2523 ci: improve travis integration conditionals snapcraft merged Merged updates to Travis CI configuration that correct spread test conditionals, preventing failures from missing Google Keys. Removes an unused cron trigger and standardizes .travis.yml indentation. Approved by one reviewer.
85%
#352 ci: add conditionals for docs-only checks imagecraft merged Merged pull request adding CI conditionals to skip checks for documentation-only changes. Approved by a reviewer, passed all CI checks, and successfully integrated into the main branch.
76%
#2217 Travis test snapcraft merged Merged after approval. Updated Travis CI configuration across 26 files, reducing 483 lines. A Codecov base commit warning appeared but did not prevent the merge.
76%
#18 ci: ensure tests succeed in CI craft-application merged Merged updates to CI configuration ensuring tests pass successfully. Approved by one reviewer, modifying five files with 23 additions and 13 deletions.
72%
#656 Use requirements files in travis tests snapcraft merged Merged changes to update Travis CI tests to use requirements files. The pull request modifies two files (+6/-4 lines) and was integrated by maintainer come-maiz without additional review comments.
72%
#143 ci: make test jobs conditional starflow merged Merged to conditionally enable CI test jobs, satisfying Charmcraft dependencies and upcoming Terraform ruleset requirements. Approved by one reviewer, passed all CI checks, and updated one file.
71%
#59 Adding travis and coverall badges snapcraft merged Merged pull request adding Travis CI and Coveralls status badges to the repository. The change added 8 lines to a single file and received reviewer approval before integration.
71%