Testing and merging pr 622 (#714)

* Add glob matching to include/exclude in diffable

* Update the README

---------

Co-authored-by: Adrian Veliz <avelizmurrieta@rccl.com>

Author: decyjphr

README.md

@@ -290,7 +290,21 @@ The following can be configured:
 - `Rulesets`
 - `Environments` - wait timer, required reviewers, prevent self review, protected branches deployment branch policy, custom deployment branch policy, variables, deployment protection rules
 
-It is possible to provide an `include` or `exclude` settings to restrict the `collaborators`, `teams`, `labels` to a list of repos or exclude a set of repos for a collaborator.
+> [!important]
+> It is possible to provide an `include` or `exclude` settings to restrict the `collaborators`, `teams`, `labels` to a list of repos or exclude a set of repos for a collaborator. 
+> The include/exclude pattern can also be for glob. For e.g.:
+```
+teams:
+  - name: Myteam-admins
+    permission: admin
+  - name: Myteam-developers
+    permission: push
+  - name: Other-team
+    permission: push
+    include:
+      - '*-config'
+```
+> Will only add `Other-team` to only `*-config` repos
 
 See [`docs/sample-settings/settings.yml`](docs/sample-settings/settings.yml) for a sample settings file.
 

lib/plugins/diffable.js

@@ -22,6 +22,7 @@
 const ErrorStash = require('./errorStash')
 const MergeDeep = require('../mergeDeep')
 const NopCommand = require('../nopcommand')
+const Glob = require('../glob')
 const ignorableFields = ['id', 'node_id', 'default', 'url']
 module.exports = class Diffable extends ErrorStash {
   constructor (nop, github, repo, entries, log, errors) {
@@ -39,7 +40,10 @@ module.exports = class Diffable extends ErrorStash {
     // this.log.debug(` entries ${JSON.stringify(filteredEntries)}`)
     filteredEntries = filteredEntries.filter(attrs => {
       if (Array.isArray(attrs.exclude)) {
-        if (!attrs.exclude.includes(this.repo.repo)) {
+        const excludeGlobs = attrs.exclude.map(exc => new Glob(exc));
+        const excludeGlobsMatch = excludeGlobs.some(glob => !!this.repo.repo.match(glob));
+
+        if (!attrs.exclude.includes(this.repo.repo) && !excludeGlobsMatch) {
           // this.log.debug(`returning not excluded entry = ${JSON.stringify(attrs)} for repo ${this.repo.repo}`)
           return true
         } else {
@@ -53,7 +57,10 @@ module.exports = class Diffable extends ErrorStash {
     })
     filteredEntries = filteredEntries.filter(attrs => {
       if (Array.isArray(attrs.include)) {
-        if (attrs.include.includes(this.repo.repo)) {
+        const includeGlobs = attrs.include.map(inc => new Glob(inc));
+        const includeGlobsMatch = includeGlobs.some(glob => !!this.repo.repo.match(glob));
+
+        if (attrs.include.includes(this.repo.repo) || includeGlobsMatch) {
           // this.log.debug(`returning included entry  = ${JSON.stringify(attrs)} for repo ${this.repo.repo}`)
           return true
         } else {