add autorust.toml with tags_allow to limit tags #747
Conversation
| @@ -0,0 +1 @@ | |||
| tags_allow = ["package-2021-08", "package-2021-05"] | |||
There was a problem hiding this comment.
This is limiting the tags for azure_mgmt_network to two of the most recent stable tags.
There was a problem hiding this comment.
So no root section is necessary? Should we have one to avoid future issues, or likely not a problem?
FWIW, our autorest tooling makes heavy use (required, really) of configuration so I definitely could see adding more in the future here...assuming we don't eventually adopt autorest instead.
| use super::*; | ||
|
|
||
| #[test] | ||
| fn test_tags_allow() -> Result<(), Error> { |
There was a problem hiding this comment.
Would be good to cover the case where no autorest.toml exists and the passed tags are returned back. That'll happen far more frequently.
| @@ -0,0 +1 @@ | |||
| tags_allow = ["package-2021-08", "package-2021-05"] | |||
There was a problem hiding this comment.
So no root section is necessary? Should we have one to avoid future issues, or likely not a problem?
FWIW, our autorest tooling makes heavy use (required, really) of configuration so I definitely could see adding more in the future here...assuming we don't eventually adopt autorest instead.
This allows specifying an allow list of tags to generate for a crate. The configuration for the package is an
autorust.tomlfile in the destination directory. We can extend it and move package specific configuration there.