Skip to content

Commit

Permalink
Merge pull request #19 from pelotech/fix/nodejs-remove-duplicate-label
Browse files Browse the repository at this point in the history
Fix/nodejs remove duplicate label
  • Loading branch information
emmanuel authored Dec 2, 2022
2 parents a778688 + 2a1187d commit 7b529e1
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 45 deletions.
42 changes: 0 additions & 42 deletions .github/workflows/lint-test.yaml

This file was deleted.

2 changes: 1 addition & 1 deletion charts/nodejs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ apiVersion: v2
appVersion: "1.0"
description: A Helm chart for Deploying NoodeJS applications to Kubernetes
name: nodejs
version: 0.2.0
version: 0.2.1
3 changes: 1 addition & 2 deletions charts/nodejs/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,9 @@ Create chart name and version as used by the chart label.
Common labels
*/}}
{{- define "nodejs.labels" -}}
app.kubernetes.io/name: {{ include "nodejs.name" . }}
helm.sh/chart: {{ include "nodejs.chart" . }}
app.kubernetes.io/instance: {{ .Release.Name }}
app.kubernetes.io/name: {{ include "nodejs.name" . }}
app.kubernetes.io/instance: {{ .Release.Name }}
{{- if (and false .Chart.AppVersion) }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- end }}
Expand Down

0 comments on commit 7b529e1

Please sign in to comment.