Fix append error in admin/run.go #46

mlamalle · 2024-11-18T11:06:55+01:00

mlamalle commented

2024-11-18 11:06:55 +01:00

Description

Cette pull request vise à résoudre un problème détecté par l'outil go vet dans le fichier internal/command/server/admin/run.go du projet Cadoles/bouncer. L'erreur spécifique est liée à l'utilisation de la fonction append sans valeurs dans le fichier run.go à la ligne 20.

L'erreur se présente comme suit :

internal/command/server/admin/run.go:20:11: append with no values

Cette erreur indique que la fonction append est utilisée sans aucun argument, ce qui n'est pas autorisé en Go.

Changements

Les changements apportés dans cette pull request concernent la correction de l'erreur détectée par go vet :

La fonction append a été remplacée par une simple affectation de la variable flags avec la valeur de common.Flags(), car il n'y avait pas de valeurs à ajouter.
Les modifications sont apportées dans le fichier internal/command/server/admin/run.go, spécifiquement dans la fonction RunCommand().

Avant :

flags := append(
    common.Flags(),
)

Après :

flags := common.Flags()

## Description Cette pull request vise à résoudre un problème détecté par l'outil `go vet` dans le fichier `internal/command/server/admin/run.go` du projet Cadoles/bouncer. L'erreur spécifique est liée à l'utilisation de la fonction `append` sans valeurs dans le fichier `run.go` à la ligne 20. L'erreur se présente comme suit : ``` internal/command/server/admin/run.go:20:11: append with no values ``` Cette erreur indique que la fonction `append` est utilisée sans aucun argument, ce qui n'est pas autorisé en Go. ## Changements Les changements apportés dans cette pull request concernent la correction de l'erreur détectée par `go vet` : - La fonction `append` a été remplacée par une simple affectation de la variable `flags` avec la valeur de `common.Flags()`, car il n'y avait pas de valeurs à ajouter. - Les modifications sont apportées dans le fichier `internal/command/server/admin/run.go`, spécifiquement dans la fonction `RunCommand()`. Avant : ```go flags := append( common.Flags(), ) ``` Après : ```go flags := common.Flags() ```

mlamalle added 1 commit 2024-11-18 11:06:56 +01:00

fix : suppression de append inutile, remonté par go vet

Cadoles/bouncer/pipeline/head This commit looks good

Details

Cadoles/bouncer/pipeline/pr-develop This commit looks good

Details

826edef358

wpetit changed title from ~~fix : suppression de append inutile, remonté par go vet~~ to Fix append error in admin/run.go

2025-03-07 10:21:15 +01:00

wpetit merged commit 076a3d784e into develop

2025-03-07 10:21:28 +01:00

wpetit deleted branch fix-append

2025-03-07 10:21:30 +01:00

wpetit referenced this issue from a commit

2025-03-07 10:21:30 +01:00

Merge pull request 'Fix append error in admin/run.go' (#46) from fix-append into develop

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Cadoles/bouncer#46