travis docker command should be simplified by setting working dir
Metadata
Current evaluation
Contributor PR snapcore/snapcraft#1011 was superseded by #1012, merged as b66bdebe9 to split Travis Docker build phases into separate docker exec commands.
Resolution: closed superseded
Reason: snapcore/snapcraft#1012 replaced the approach proposed in #1011 and merged as commit b66bdebe9, implementing clearer, separated Docker build phases for Travis.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Currently the default proposed
apt update -qq && apt install snapcraft -y && cd $(pwd) && snapcraft
I'd personally prefer also to use a different form with multiple line commands, by using a "builder"-named docker instance and calling it in multiple lines, as it provides better results in logs and it's way more readable.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Contributor PR snapcore/snapcraft#1011 was superseded by #1012, merged as b66bdebe9 to split Travis Docker build phases into separate docker exec commands. | |
| stealth/space-bunny-alpha | — | closed stale | Travis Docker cleanup was proposed in snapcore/snapcraft#1011, but the PR was closed unmerged as stale in March 2017. | |
| stealth/space-bunny-alpha | — | closed stale | The proposed Travis Docker cleanup landed in PR #1011, which was closed as stale in March 2017 without being merged. | |
| stealth/space-bunny-alpha | — | closed stale | The Travis Docker working-directory and multiline-command proposal was abandoned without a linked PR, commit, or documented resolution. | |
| qwen/qwen3.6-35b-a3b | — | — | The proposal to simplify the Travis Docker command using a named builder and multi-line formatting was closed without implementation. With zero comments and no resolution, the suggestion was abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation due to inactivity. The proposal to simplify the Travis Docker command by setting the working directory and using multi-line builder commands was abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation. The suggestion to simplify the Travis CI docker command using a working directory and multi-line builder steps was abandoned or superseded by updated CI configurations. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Simplified Travis CI Docker command suggestions were closed without implementation or discussion. No build configuration changes were applied, and the proposal was abandoned. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
snapcore/snapcraft#1011
(confidence 100%)
Original contributor PR linked from the issue before the maintainer replacement.
-
Superseded By:
snapcore/snapcraft#1012
(confidence 98%)
Merged as b66bdebe9 with the Travis docker exec build-phase refactor.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1012 travis.yaml: use docker exec to split build phases | snapcraft | merged | 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). | |
| #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. |