Skip to content

allow to save a Docker deps image for PR having [dockerdeps] in their message#6516

Merged
3nids merged 2 commits intoqgis:masterfrom
3nids:pr_push_deps
Mar 6, 2018
Merged

allow to save a Docker deps image for PR having [dockerdeps] in their message#6516
3nids merged 2 commits intoqgis:masterfrom
3nids:pr_push_deps

Conversation

@3nids
Copy link
Copy Markdown
Member

@3nids 3nids commented Mar 2, 2018

keyword is: [ dockerdeps ] without the spaces

@3nids 3nids force-pushed the pr_push_deps branch 2 times, most recently from c968e06 to c615182 Compare March 2, 2018 19:06
@3nids 3nids changed the title allow to save a Docker deps image for PR having keyword in their message allow to save a Docker deps image for PR having [dockerdeps] in their message Mar 2, 2018
@3nids 3nids closed this Mar 2, 2018
@3nids 3nids reopened this Mar 2, 2018
@3nids 3nids force-pushed the pr_push_deps branch 3 times, most recently from 079eb56 to 6efe1b8 Compare March 2, 2018 19:32
@3nids
Copy link
Copy Markdown
Member Author

3nids commented Mar 6, 2018

sadly, Travis doesn't know about the PR message but only about commit message.
As for now, this will push the deps images if the last commit of the Travis build has [dockerdeps] in its title.
I filled a feature request for this on Travis
travis-ci/travis-ci#9288
and tried to PR for this
travis-ci/travis-scheduler#131
travis-ci/travis-build#1319
without much luck

@3nids 3nids merged commit ec975ba into qgis:master Mar 6, 2018
@3nids 3nids deleted the pr_push_deps branch March 6, 2018 13:32
@3nids
Copy link
Copy Markdown
Member Author

3nids commented Mar 7, 2018

seems to cause issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant