Skip to content

Improve Travis CI deploy/send PR condition - #1136

Merged
PeterDaveHello merged 1 commit into
nodejs:masterfrom
PeterDaveHello:update-travis-ci
Oct 26, 2019
Merged

PeterDaveHello merged 1 commit into
nodejs:masterfrom
PeterDaveHello:update-travis-ci

Conversation

@PeterDaveHello

Copy link
Copy Markdown
Member

Originally it supports only Dockerfile changes, with this adjustment,
it'll now send pull request to the downstream for the version and tag
array in generate-stackbrew-library.sh file.

cc #1134

Comment thread travis.yml.template
Originally it supports only Dockerfile changes, with this adjustment,
it'll now send pull request to the downstream for the version and tag
array in generate-stackbrew-library.sh file.
@PeterDaveHello

Copy link
Copy Markdown
Member Author

I think I found a bug before merging, need another review, thanks.

@LaurentGoderre

Copy link
Copy Markdown
Member

Im wondering if it would be safer to compare the output of stackbrew with upstream instead of changes to the script because changes to the config could result in a different stackbrew without changing the script itself.

@PeterDaveHello

PeterDaveHello commented Oct 23, 2019 •

Copy link
Copy Markdown
Member Author

Could do that in the future, the enhancement here actually follows the tag / version array in the script only.

@PeterDaveHello
PeterDaveHello merged commit 1e228b0 into nodejs:master Oct 26, 2019
@PeterDaveHello
PeterDaveHello deleted the update-travis-ci branch October 26, 2019 19:36
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.

2 participants