search

xoac / rust-azure-pipelines

Collection of azure pipelines templates for rust
MIT License
44 stars 12 forks source link

Fix test-flags that should be test_flags #20

Closed russelltg closed 2 years ago

russelltg commented 4 years ago

This had me scratching my head for quite a while when I added options here.

If you look in ci/jobs/cargo-test.yaml, it is test_flags and not test-flags.

russelltg commented 4 years ago

Unless I have a misconecption about how this all works, I'm kinda surprised that the tests passed.

It seems like parameters' defaults aren't deeply overidden--as in if you specify any test_flags you loose the default and the whole mapping gets removed.

xoac commented 2 years ago

Thanks for contribution and sorry for not having time to review and merge it. I archive this repo for now.