Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explain how Docker Hub factors in the workflow. #171

Merged
merged 1 commit into from
Dec 13, 2021

Conversation

rosiel
Copy link
Contributor

@rosiel rosiel commented Dec 13, 2021

Proposing wording that explains Docker Hub's role in bridging Buildkit and Isle-DC.

Thanks to @adam-vessey and @noahwsmith for answering my question on Slack.

Copy link
Contributor

@nigelgbanks nigelgbanks left a comment

Choose a reason for hiding this comment

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

Looks good, I've taken a look at the failed build seems to be a network issue, I'm rerunning and will sort out the issue if its something else.

@nigelgbanks
Copy link
Contributor

Looks to be something else, for the failed build. Regardless it's not related to this change I'll investigate that separately.

@nigelgbanks nigelgbanks merged commit a4ddee3 into main Dec 13, 2021
@nigelgbanks nigelgbanks deleted the rosiel-explain-dockerhub branch December 13, 2021 20:38
rosiel added a commit that referenced this pull request Jan 5, 2022
* Fix logging errors with cantaloupe due to find/replace bug (#165)

* Adding max_input_vars to php.ini (#166)

* Explain how Docker Hub factors in the workflow. (#171)

* Updated the introduction to redirect users to better starting points in case they ended up at this repo by accident. (#164)

* [issue-172] Use explicit commits in URLs such that (#176)

Code can change when pulling from a master branch, which causes checksum
checks to fail.

This can changes master to instead refer to explicit commits hashes.

The output docker images is byte for byte the same though.

* Prevent passwords from being logged.

Co-authored-by: dannylamb <dlamb@islandora.ca>
Co-authored-by: Rosie Le Faive <lefaive@gmail.com>
rosiel added a commit that referenced this pull request Feb 10, 2022
* Fix logging errors with cantaloupe due to find/replace bug (#165)

* Adding max_input_vars to php.ini (#166)

* Explain how Docker Hub factors in the workflow. (#171)

* Updated the introduction to redirect users to better starting points in case they ended up at this repo by accident. (#164)

* [issue-172] Use explicit commits in URLs such that (#176)

Code can change when pulling from a master branch, which causes checksum
checks to fail.

This can changes master to instead refer to explicit commits hashes.

The output docker images is byte for byte the same though.

* Prevent passwords from being logged.

Co-authored-by: dannylamb <dlamb@islandora.ca>
Co-authored-by: Rosie Le Faive <lefaive@gmail.com>
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