Skip to content
This repository was archived by the owner on Jan 12, 2022. It is now read-only.

Docker Builds#9

Merged
hrpatel merged 84 commits intomasterfrom
docker_build
Sep 12, 2018
Merged

Docker Builds#9
hrpatel merged 84 commits intomasterfrom
docker_build

Conversation

@hrpatel
Copy link
Copy Markdown
Contributor

@hrpatel hrpatel commented Sep 6, 2018

No description provided.

@hrpatel hrpatel added the WIP label Sep 6, 2018
@hrpatel hrpatel self-assigned this Sep 6, 2018
Copy link
Copy Markdown

@antevens antevens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hate this UI

Comment thread bin/travis-ci/check_tag.sh Outdated
Comment thread bin/travis-ci/check_tag.sh Outdated
Comment thread bin/travis-ci/check_tag.sh Outdated
Comment thread bin/travis-ci/check_tag.sh Outdated
sudo curl -s -L -o /usr/local/bin/shtdlib.sh https://github.com/sdelements/shtdlib/raw/master/shtdlib.sh
sudo chmod 775 /usr/local/bin/shtdlib.sh
# shellcheck disable=SC1091
source /usr/local/bin/shtdlib.sh
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably better to provide a hint where to find it for shellcheck

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eh?

Comment thread bin/travis-ci/check_tag.sh Outdated
Comment thread bin/travis-ci/deb_build.sh
Comment thread LICENSE Outdated
Comment thread bin/travis-ci/check_tag.sh Outdated
Comment thread bin/travis-ci/deb_package.sh Outdated
Comment thread bin/travis-ci/docker_publish.sh Outdated
Copy link
Copy Markdown

@adamgilbert adamgilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 if it works and you're happy with it!

@hrpatel hrpatel merged commit dd3f8a6 into master Sep 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants