[ci] Allow + in Bazel target names

This has come up a couple of times recently with names containing `C++`
which are allowed by Bazel but banned by this rule.

I don't think a `+` could cause the kind of type confusion that the
comment here talks about. Also Bazel 8 uses `+` in canonical target
names so we may need to accept them anyway if `query` ever spits out
one of those.

Signed-off-by: James Wainwright <[email protected]>

Author: jwnrt

ci/scripts/lib/bazel_query.py

@@ -57,7 +57,7 @@ def find_targets_with_banned_chars(self) -> Iterable[Tuple[str, Set[str]]]:
           set of characters that must be removed from the target's name.
 
         """
-        allowed_chars = set(string.ascii_letters + string.digits + '/:_-.')
+        allowed_chars = set(string.ascii_letters + string.digits + '/:_-.+')
         for target in self.query("//..."):
             if bad_chars := set(target) - allowed_chars:
                 yield (target, bad_chars)