Skip to content

Commit 533b657

Browse files
author
Taavi Ansper
committed
Merge branch 'issue/43-dashes-removed-from-group-names' into 'master'
Trying new slug patterns for more recent Gitlab group name and slug rules. See merge request sysadm/gitlab-ce-ldap-sync!1
2 parents 0fdd389 + 98e8c61 commit 533b657

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/LdapSyncCommand.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1296,14 +1296,14 @@ private function deployGitlabUsersAndGroups(array $config, string $gitlabInstanc
12961296
$this->output?->writeln(sprintf("Deploying users and groups to Gitlab instance \"%s\"...", $gitlabInstance));
12971297

12981298
$slugifyGitlabName = new Slugify([
1299-
"regexp" => "/([^A-Za-z0-9]|-_\. )+/",
1299+
"regexp" => "/([^A-Za-z0-9_\.\(\)\- ])+/",
13001300
"separator" => " ",
13011301
"lowercase" => false,
13021302
"trim" => true,
13031303
]);
13041304

13051305
$slugifyGitlabPath = new Slugify([
1306-
"regexp" => "/([^A-Za-z0-9]|-_\.)+/",
1306+
"regexp" => "/([^A-Za-z0-9_\.\-])+/",
13071307
"separator" => "-",
13081308
"lowercase" => true,
13091309
"trim" => true,

0 commit comments

Comments
 (0)