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

(DRON-113) use ghodss/yaml for yaml printing #189

Merged
merged 1 commit into from
Aug 20, 2021

Conversation

tphoney
Copy link
Contributor

@tphoney tphoney commented Aug 16, 2021

➜  drone-cli git:(drone-113) ✗ release/linux/amd64/drone jsonnet --source=bla.jsonnet  --format --stdout=true
kind: pipeline
name: default
platform:
  arch: amd64
  os: linux
  prune: false
steps:
- image: alpine:latest
  name: test
  pull: always
  settings:
    config: policy.yml
    project: test-dev
    prune: false
    set_env_vars:
      prune: false

and stream with multi-part documents with stream

release/linux/amd64/drone jsonnet --source=multi.jsonnet --stream --format --stdout=true
---
kind: pipeline
name: go-1-11
steps:
- commands:
  - go get
  - go test
  image: golang:1.11
  name: test
---
kind: pipeline
name: go-1-12
steps:
- commands:
  - go get
  - go test
  image: golang:1.12
  name: test

And starlark

➜  drone-cli git:(drone-113) ✗ release/linux/amd64/drone starlark --source=bla.star  --format --stdout=true
kind: pipeline
name: build
steps:
- commands:
  - echo hello world
  image: alpine
  name: build

@tphoney tphoney changed the title (DRON-113) use ghodss/yaml for yaml pretty print (DRON-113) use ghodss/yaml for yaml printing Aug 16, 2021
@tphoney tphoney force-pushed the drone-113 branch 2 times, most recently from e4853b6 to 47e6978 Compare August 16, 2021 18:44
@tphoney
Copy link
Contributor Author

tphoney commented Aug 17, 2021

@bradrydzewski changes made, thanks for your patience :)

@tphoney tphoney merged commit 338ba6d into harness:master Aug 20, 2021
@tphoney tphoney added the bug label Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants