Skip to content

Commit 62df514

Browse files
committed
Remove unnecessary separate API call for repo topics
Signed-off-by: Timo Sand <[email protected]>
1 parent 8359c39 commit 62df514

1 file changed

Lines changed: 0 additions & 25 deletions

File tree

github/resource_github_repository.go

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -756,14 +756,6 @@ func resourceGithubRepositoryCreate(ctx context.Context, d *schema.ResourceData,
756756
d.SetId(repo.GetName())
757757
}
758758

759-
topics := repoReq.Topics
760-
if len(topics) > 0 {
761-
_, _, err := client.Repositories.ReplaceAllTopics(ctx, owner, repoName, topics)
762-
if err != nil {
763-
return diag.FromErr(err)
764-
}
765-
}
766-
767759
pages := expandPages(d.Get("pages").([]any))
768760
if pages != nil {
769761
_, _, err := client.Repositories.EnablePages(ctx, owner, repoName, pages)
@@ -995,23 +987,6 @@ func resourceGithubRepositoryUpdate(ctx context.Context, d *schema.ResourceData,
995987
}
996988
}
997989

998-
if d.HasChange("topics") {
999-
topics := repoReq.Topics
1000-
_, _, err = client.Repositories.ReplaceAllTopics(ctx, owner, *repo.Name, topics)
1001-
if err != nil {
1002-
return diag.FromErr(err)
1003-
}
1004-
d.SetId(*repo.Name)
1005-
1006-
if d.HasChange("topics") {
1007-
topics := repoReq.Topics
1008-
_, _, err = client.Repositories.ReplaceAllTopics(ctx, owner, *repo.Name, topics)
1009-
if err != nil {
1010-
return diag.FromErr(err)
1011-
}
1012-
}
1013-
}
1014-
1015990
if d.HasChange("vulnerability_alerts") {
1016991
err = updateVulnerabilityAlerts(d, client, ctx, owner, repoName)
1017992
if err != nil {

0 commit comments

Comments
 (0)