Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Egonz/auto 457 1.8 #1048

Merged
merged 9 commits into from
Feb 5, 2021
Merged

Egonz/auto 457 1.8 #1048

merged 9 commits into from
Feb 5, 2021

Conversation

cocho-gore
Copy link
Contributor

Change Description

From AUTO-457, we are working on providing the EOSIO community minimal docker images via Docker Hub which they can pull to more easily use our tooling in a clean environment. We have already introduced a docker image for EOSIO, so this pull request creates one for EOSIO.CDT.

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

Public docker images are now available at docker.io/eosio/eosio.cdt.

Copy link

@zach-cpu zach-cpu left a comment

Choose a reason for hiding this comment

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

Rename to Dockerfile for consistency.

.cicd/create-docker-from-binary.sh Show resolved Hide resolved
scottarnette
scottarnette previously approved these changes Feb 5, 2021
zach-cpu
zach-cpu previously approved these changes Feb 5, 2021
Copy link
Contributor

@kj4ezj kj4ezj left a comment

Choose a reason for hiding this comment

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

Since we're making these other changes anyways, I'd also recommend removing the blank lines. They serve no purpose. But, that is up to you.

.cicd/pipeline.yml Outdated Show resolved Hide resolved
docker/dockerfile Outdated Show resolved Hide resolved
@cocho-gore cocho-gore dismissed stale reviews from zach-cpu and scottarnette via e718159 February 5, 2021 19:14
Copy link
Contributor

@kj4ezj kj4ezj left a comment

Choose a reason for hiding this comment

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

🐳

@kj4ezj kj4ezj merged commit 9da030a into release/1.8.x Feb 5, 2021
@kj4ezj kj4ezj deleted the egonz/AUTO-457-1.8 branch February 5, 2021 22:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants