← Back to issue list

travis.yaml: use docker exec to split build phases

View original Github issue

Metadata

Project
snapcraft
Number
#1012
Type
pull request
State
merged
Author
3v1n0
Labels
—
Created
Updated
Closed

Current evaluation

PR #1012 by 3v1n0 merged as commit b66bdebe9 (Dec 2016): .travis.yaml reworked to run build phases via `docker exec` lines for readability (+11/-4).

Resolution: closed resolved

Reason: The pull request itself was merged into master as commit b66bdebe9 ("travis.yaml: use docker exec to split build phases (#1012)"), so the proposed .travis.yaml restructuring shipped; only a Codecov coverage bot comment was left on it.

Evaluated by: mr-cal-bot

No scores available.

Issue body

We can do things in multiple lines keeping things more readable and clean

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #1012 by 3v1n0 merged as commit b66bdebe9 (Dec 2016): .travis.yaml reworked to run build phases via `docker exec` lines for readability (+11/-4).
stealth/space-bunny-alpha — closed resolved Merged PR #1012 in commit b66bdebe9: Travis build phases were split with docker exec for clearer, more readable configuration.
qwen/qwen3.6-35b-a3b — — Merged updates to the Travis config using docker exec to split build phases, improving readability. The change modified one file with 11 additions and 4 deletions. Codecov confirmed 96.33% coverage with 100% diff coverage.
qwen3.6-35b-a3b-mtp-q6 — — Merged an external update to .travis.yml that uses docker exec to split build phases, improving script readability. The change maintained 96.33% overall code coverage with 100% diff coverage.

Update history

No update history recorded yet.

Related work

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

    Later travis change using the snapcore/snapcraft docker image, a descendant of this CI restructuring.

Related issues

Issue Project State Summary Similarity
#1659372 travis docker command should be simplified by setting working dir snapcraft (launchpad) closed Contributor PR snapcore/snapcraft#1011 was superseded by #1012, merged as b66bdebe9 to split Travis Docker build phases into separate docker exec commands.
79%
#1653685 clean up travis.yaml snapcraft (launchpad) closed Resolved by merged snapcore/snapcraft PR #1012 (commit b66bdebe9), which cleaned up Travis and changed container build phases to use docker exec.
74%