Simplify glob pattern

[trask]

Follow-up to @chalin's #2548 (comment)

[chalin]

Actually, what I was alluding to earlier is that foo/**/** could be simplified to foo/** to match the branches that I think we want to match.

It worked in the OTel.io settings:

Note the match given the pattern opentelemetrybot/**.

According to the GH docs, https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-rulesets/creating-rulesets-for-a-repository#using-fnmatch-syntax. The base docs of fnmatch are https://ruby-doc.org/core-2.5.1/File.html#method-c-fnmatch.

According to Copilot:

When applied to branch names, the ** pattern in fnmatch matches substrings recursively, treating slashes as ordinary characters.

The quote above supports my claim that ** should be enough. WDYT?

[trask]

I tried otelbot/** but it didn't match a branch named otelbot/one/two

[chalin]

I tried otelbot/** but it didn't match a branch named otelbot/one/two

Yeah, I can see that.

Funny: to me abc/**/* should not match abc/def because of the two slashes in the pattern; but I guess I'm reading this too much like a regex rather than a (file) glob pattern -- even though branch names aren't path names.

[chalin]

👍🏻