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

feat: Support YAML as Package param #1350

Merged
merged 12 commits into from Sep 20, 2023
Merged

feat: Support YAML as Package param #1350

merged 12 commits into from Sep 20, 2023

Conversation

victorcolombo
Copy link
Contributor

Description:

Add support for providing YAML instead of JSON in both SDK and CLI:
kurtosis run github.com/kurtosis-tech/datastore-army-package 'num_datastores: 2 # THIS IS A COMMENT'

Is this change user facing?

YES

References (if applicable):

Closes #581

@victorcolombo victorcolombo enabled auto-merge (squash) September 20, 2023 21:41
@victorcolombo victorcolombo merged commit e33bfe6 into main Sep 20, 2023
30 checks passed
@victorcolombo victorcolombo deleted the vcolombo/gh-581 branch September 20, 2023 23:41
h4ck3rk3y pushed a commit that referenced this pull request Sep 21, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.83.5](0.83.4...0.83.5)
(2023-09-21)


### Features

* add service details to EM UI
([#1352](#1352))
([2ccd98d](2ccd98d))
* added ability to lint Starlark packages
([#1360](#1360))
([f4a072c](f4a072c)),
closes [#1228](#1228)
* Support YAML as Package param
([#1350](#1350))
([e33bfe6](e33bfe6))


### Bug Fixes

* tail logs from end of log file
([#1339](#1339))
([b8d5816](b8d5816))
* warn instead of failing for json log line parse error
([#1336](#1336))
([44b2820](44b2820))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
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.

args should be specifiable as YAML
2 participants