From cb7f811bf6c6c16bcaffdcdde8be4e68c65babdb Mon Sep 17 00:00:00 2001 From: Jan Staelens Date: Mon, 2 Dec 2024 16:15:42 +0100 Subject: [PATCH] Extra validation check for manifest tags (#11) (#12) Co-authored-by: Arne Maes --- .../CatalogManager.cs | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Skyline.DataMiner.CICD.Tools.GitHubToCatalogYaml/CatalogManager.cs b/Skyline.DataMiner.CICD.Tools.GitHubToCatalogYaml/CatalogManager.cs index 6767a7f..10862ad 100644 --- a/Skyline.DataMiner.CICD.Tools.GitHubToCatalogYaml/CatalogManager.cs +++ b/Skyline.DataMiner.CICD.Tools.GitHubToCatalogYaml/CatalogManager.cs @@ -184,13 +184,19 @@ private async Task CheckTags(CatalogYaml catalogYaml) catalogYaml.Tags = new List(); } + if(catalogYaml.Tags.Count >= 5) + { + logger.LogDebug("Catalog YAML has the max amount of tags already. Skipping the adding GitHub topics step."); + return; + } + var topics = await service.GetRepositoryTopicsAsync(); if (topics is { Count: > 0 }) { catalogYaml.Tags.AddRange(topics); // Remove duplicates - catalogYaml.Tags = catalogYaml.Tags.Distinct().ToList(); + catalogYaml.Tags = catalogYaml.Tags.Distinct().Take(5).ToList(); logger.LogDebug("Distinct GitHub Topics found and applied."); } }