Skip to content

chore: remove use of deprecated docker-compose - #137

Merged
strophy merged 1 commit into
18.0.0from
remove-compose-dep
Dec 6, 2022
Merged

chore: remove use of deprecated docker-compose#137
strophy merged 1 commit into
18.0.0from
remove-compose-dep

Conversation

@strophy

Copy link
Copy Markdown
Collaborator

This PR removes the docker-compose requirement. Docker Compose V1 (Python) has been deprecated and replaced with Docker Compose V2 (Go) which now ships by default with both Docker Desktop and the convenience script we use to install Docker on Linux. This PR can be merged when support for Docker Compose V2 is merged into Dashmate (dashevo/platform #422).

@strophy
strophy requested a review from thephezJune 10, 2022 05:25

@thephezthephez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

utACK 👍

@strophy
strophy marked this pull request as ready for review July 13, 2022 07:05
@strophy
strophy marked this pull request as draft July 13, 2022 07:15
@strophy

Copy link
Copy Markdown
CollaboratorAuthor

Docker Compose V2 will be supported in Platform 0.23, so we need to wait for that release to merge this now.

@thephez
thephez changed the base branch from 0.17.0 to 18.0.0August 18, 2022 12:16
@thephez

Copy link
Copy Markdown
Collaborator

@strophy Are we waiting for the actual v0.23 release? Or is this able to be merged now?

@strophy

Copy link
Copy Markdown
CollaboratorAuthor

We're still waiting for release afaik, but I think we could merge this already since testnet is already running v0.23

@strophy
strophy marked this pull request as ready for review December 6, 2022 08:03
@strophy
strophy merged commit a55b8ae into 18.0.0Dec 6, 2022
@strophy
strophy deleted the remove-compose-dep branch December 6, 2022 08:06
Sign up for freeto 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

@strophy@thephez