Skip to content

Commit

Permalink
chore: remove unused variables in GenerateManifests (#13887)
Browse files Browse the repository at this point in the history
Signed-off-by: Abirdcfly <[email protected]>
  • Loading branch information
Abirdcfly authored Jun 2, 2023
1 parent ec2340a commit d7cc8da
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions reposerver/repository/repository.go
Original file line number Diff line number Diff line change
Expand Up @@ -1317,7 +1317,6 @@ func WithCMPTarExcludedGlobs(excludedGlobs []string) GenerateManifestOpt {
func GenerateManifests(ctx context.Context, appPath, repoRoot, revision string, q *apiclient.ManifestRequest, isLocal bool, gitCredsStore git.CredsStore, maxCombinedManifestQuantity resource.Quantity, gitRepoPaths io.TempPaths, opts ...GenerateManifestOpt) (*apiclient.ManifestResponse, error) {
opt := newGenerateManifestOpt(opts...)
var targetObjs []*unstructured.Unstructured
var dest *v1alpha1.ApplicationDestination

resourceTracking := argo.NewResourceTracking()

Expand Down Expand Up @@ -1417,15 +1416,10 @@ func GenerateManifests(ctx context.Context, appPath, repoRoot, revision string,
}
}

res := apiclient.ManifestResponse{
return &apiclient.ManifestResponse{
Manifests: manifests,
SourceType: string(appSourceType),
}
if dest != nil {
res.Namespace = dest.Namespace
res.Server = dest.Server
}
return &res, nil
}, nil
}

func newEnv(q *apiclient.ManifestRequest, revision string) *v1alpha1.Env {
Expand Down

0 comments on commit d7cc8da

Please sign in to comment.