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

Fix #552: Pin docker-compose dependency version to 0.23.6 for CLI #560

Merged
merged 2 commits into from
Aug 11, 2021

Conversation

CherryDT
Copy link
Contributor

@CherryDT CherryDT commented Aug 7, 2021

Context

#552: Error when creating database snapshot

It's not possible to create a database snapshot because it fails with Cannot read property 'trim' of undefined.

The issue exists because of a breaking change (imo a bug) introduced in [email protected] (PDMLab/docker-compose#178) which crashes when ps is used with -q.

Proposed Solution

Pin version of docker-compose to 0.23.6.

Once this issue is resolved upstream, it can be updated to a more relaxed semver again that points to versions that include the fix.

@rhyslbw rhyslbw self-requested a review August 9, 2021 04:23
Copy link
Contributor

@rhyslbw rhyslbw left a comment

Choose a reason for hiding this comment

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

Thanks for debugging this @CherryDT

@rhyslbw rhyslbw linked an issue Aug 9, 2021 that may be closed by this pull request
3 tasks
@rhyslbw rhyslbw self-requested a review August 9, 2021 04:28
Copy link
Contributor

@rhyslbw rhyslbw left a comment

Choose a reason for hiding this comment

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

This repo uses a Yarn offline cache, and therefore changes to the old package to be removed, and the new added in the commit. Please squash the changes into a single commit and force push to this branch.

@CherryDT
Copy link
Contributor Author

CherryDT commented Aug 9, 2021

Sorry I don't quite understand. I did only one commit. What is it that I need to do exactly?

@rhyslbw
Copy link
Contributor

rhyslbw commented Aug 9, 2021

You should observe the new version replace the old version ./packages-cache/. This should be checked into source control

@CherryDT
Copy link
Contributor Author

CherryDT commented Aug 9, 2021

Ah, I understand now. Done.

@rhyslbw rhyslbw merged commit ab63d79 into cardano-foundation:master Aug 11, 2021
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.

@cardano-graphql/cli - TypeError: Cannot read property 'trim' of undefined
2 participants