Description
Tables' ci.yml was the only one that was inconsistent. All other instances within the repo have the LiveTestCtestRegex
value match CtestRegex
, so we fixed it in #6223, for consistency.
That said, some use wild-card (*
), while others have it wrapped in quotes (""
), while the rest leave it as plain-text:
https://github.com/search?q=repo%3AAzure%2Fazure-sdk-for-cpp%20LiveTestCtestRegex&type=code
We should consider picking a viable format, and apply it consistently, to avoid confusion.
A strawman proposal is to have it be unquoted, without the wild card, matching what we have in our template and core project:
azure-template
azure-core
azure-security-keyvault
azure-storage
Open question:
Would it make a difference to the set of tests that get matched/run if the regex value ended with a period .
or not?
azure-sdk-for-cpp/sdk/template/ci.yml
Line 29 in 384552a
azure-sdk-for-cpp/sdk/keyvault/ci.yml
Lines 30 to 31 in 384552a
azure-sdk-for-cpp/sdk/storage/ci.yml
Lines 30 to 33 in 384552a