Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Update sawtooth-default-poet.yaml #2434

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bmoore06
Copy link

@bmoore06 bmoore06 commented Sep 8, 2022

Signed-off-by: Benjamin Moore 74261880+bmoore06@users.noreply.github.com

Signed-off-by: Benjamin Moore <74261880+bmoore06@users.noreply.github.com>
Copy link

@scealiontach scealiontach left a comment

Choose a reason for hiding this comment

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

These are mostly just formatting/style changes. I'd suggest that if we are going to update it then we should make it significantly easier to read and manage. What we've got now is a gnarly mess of escaping and quotes. We could improve it by changing the commands to follow a form along the lines of

service-0:
  entrypoint:
    - bash
    - -ec
  command: |
    echo "Now we don't need to escape or use && as much";
    tail -f /dev/null

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants