fixed exclude org race condition (#393)

This commit is contained in:
Andreas Wachter
2026-05-14 19:11:11 +02:00
committed by GitHub
parent d3fd3e4717
commit 97c1d192bd
7 changed files with 51 additions and 45 deletions
+4 -4
View File
@@ -125,6 +125,10 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if include[r.Name] {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -141,10 +145,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(include) == 0 {
repos = append(repos, types.Repo{
Name: r.Name,
+10 -8
View File
@@ -322,6 +322,11 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
if time.Since(r.Updated) > repo.Filter.LastActivityDuration && repo.Filter.LastActivityDuration != 0 {
continue
}
if exclude[r.Name] {
continue
}
if include[r.Name] {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -354,10 +359,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(repo.Include) == 0 {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -468,6 +469,11 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
if time.Since(r.Updated) > repo.Filter.LastActivityDuration && repo.Filter.LastActivityDuration != 0 {
continue
}
if exclude[r.Name] {
continue
}
if include[r.Name] {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -500,10 +506,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(repo.Include) == 0 {
repos = append(repos, types.Repo{
Name: r.Name,
+8 -8
View File
@@ -264,6 +264,14 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[*r.Name] {
continue
}
if excludeorgs[r.GetOwner().GetLogin()] {
continue
}
if include[*r.Name] {
repos = append(repos, types.Repo{
Name: r.GetName(),
@@ -286,14 +294,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[*r.Name] {
continue
}
if excludeorgs[r.GetOwner().GetLogin()] {
continue
}
if len(repo.Include) == 0 {
if len(includeorgs) > 0 {
if includeorgs[r.GetOwner().GetLogin()] {
+12 -10
View File
@@ -269,6 +269,12 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
if time.Since(*r.LastActivityAt) > repo.Filter.LastActivityDuration && repo.Filter.LastActivityDuration != 0 {
continue
}
if exclude[r.Name] {
continue
}
if excludeorgs[r.Namespace.FullPath] {
continue
}
if include[r.Name] {
if r.RepositoryAccessLevel != gitlab.DisabledAccessControl {
repos = append(repos, types.Repo{
@@ -305,9 +311,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(include) == 0 {
if r.RepositoryAccessLevel != gitlab.DisabledAccessControl {
repos = append(repos, types.Repo{
@@ -436,6 +439,12 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if excludeorgs[r.Namespace.FullPath] {
continue
}
if include[r.Name] {
if r.RepositoryAccessLevel != gitlab.DisabledAccessControl {
repos = append(repos, types.Repo{
@@ -472,13 +481,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if excludeorgs[r.Namespace.FullPath] {
continue
}
if len(include) == 0 {
if len(includeorgs) == 0 || includeorgs[r.Namespace.FullPath] {
if r.RepositoryAccessLevel != gitlab.DisabledAccessControl {
+9 -8
View File
@@ -182,6 +182,11 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
if time.Since(r.Updated) > repo.Filter.LastActivityDuration && repo.Filter.LastActivityDuration != 0 {
continue
}
if exclude[r.Name] {
continue
}
if include[r.Name] {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -214,10 +219,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(include) == 0 {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -309,6 +310,10 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if include[r.Name] {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -341,10 +346,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(repo.Include) == 0 {
repos = append(repos, types.Repo{
Name: r.Name,
+4 -3
View File
@@ -95,13 +95,14 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
}
if exclude[r.Name] {
continue
}
if len(repo.Include) > 0 {
if !include[r.Name] {
continue
}
if exclude[r.Name] {
continue
}
}
defaultbranch, _, err := client.GetDefaultBranch(r.ID)
+4 -4
View File
@@ -306,6 +306,10 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
ownerName := strings.TrimPrefix(ownerCanonicalName, "~")
isPrivate := strings.EqualFold(r.Visibility, "PRIVATE") || strings.EqualFold(r.Visibility, "private")
if exclude[r.Name] {
continue
}
if include[r.Name] {
repos = append(repos, types.Repo{
Name: r.Name,
@@ -335,10 +339,6 @@ func Get(conf *types.Conf) ([]types.Repo, bool) {
continue
}
if exclude[r.Name] {
continue
}
if len(include) == 0 {
repos = append(repos, types.Repo{
Name: r.Name,