Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[gen-manifest] Support passing any sample key/value pairs #70

Closed
beccasaurus opened this issue Jun 12, 2019 · 2 comments
Closed

[gen-manifest] Support passing any sample key/value pairs #70

beccasaurus opened this issue Jun 12, 2019 · 2 comments
Assignees
Labels
🚨 This issue needs some love. triage me I really want to be triaged.

Comments

@beccasaurus
Copy link

beccasaurus commented Jun 12, 2019

The new manifest no longer has a top-level --path/--bin/etc

Let's update gen-manifest so that it can set any key/value for a sample.

Because gen-manifest creates N samples, it's good if each --key/value pair is set on EVERY sample.

So, passing --path X results in EVERY sample in the manifest having path: X

@googleapis/samplegen

@beccasaurus
Copy link
Author

@vchudnov-g

@beccasaurus beccasaurus changed the title [gen-manifest] path: should be (a) relative (b) configurable via --path [gen-manifest] Support passing any sample key/value pairs Jun 12, 2019
@beccasaurus
Copy link
Author

Important: please continue to support all existing flags until we deprecate the old schema

We have tools that pass --bin --env --invocation --chdir etc

They can just be shorthand for setting those key/values :)

vchudnov-g added a commit that referenced this issue Jun 20, 2019
…ary labels (#74)

* Add tests to existing gen-manifest functionality
* Allow gen-manifest to generate manifests conforming to schema v3 (schema v2 remains the default)
* When generating in schema v3, allow the manifest to be in a factored form (with common fields in a YAML anchor block referenced from the manifest list) or in a flat form (each list item repeats the common fields). This is controlled by the flag `--flat` (default is `false`, ie. factored output for similarity with how schema v2 was generating)
* For both schema v2 and schema v3, allow passing arbitrary --KEY=VALUE pairs on the command line, which are interpreted as tags that apply to all elements (addresses #70)
  * Forbid certain KEY names (`sample`, `path`, and sometimes `basepath`) that we autogenerate based on the sample location/contents
* Tests for the above
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. triage me I really want to be triaged. labels Apr 7, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🚨 This issue needs some love. triage me I really want to be triaged.
Projects
None yet
Development

No branches or pull requests

3 participants