Code review comment for ~kissiel/plainbox-provider-checkbox:automated-switch-gpu-job

Revision history for this message
Maciej Kisielewski (kissiel) wrote :

> It's hard to see from the various places it could be getting globbed. Are you
> able to confirm that it doesn't match existing patterns to include it
> automatically? As long as we aren't actually using this in the automated test
> plans, +1.

This is why I added _auto_ as an infix. Placing it at the end would match to what TPs are using -
 e.g.: 2_switch_card_.*

There are many providers 'out there', I checked the checkbox-dev owned ones and OEM. If the accidental inclusion happens somewhere else, I will ask for forgiveness ;-)

« Back to merge proposal