-
Notifications
You must be signed in to change notification settings - Fork 620
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use the new chart GitHub repo URL instead of the helm repository URL? There won't be much discoverable by going to https://blakeblackshear.github.io/blakeshome-charts/
charts/frigate/README.md
Outdated
``` | ||
|
||
Read through the [values.yaml](https://github.com/k8s-at-home/charts/blob/master/charts/frigate/values.yaml) file. It has several commented out suggested values. | ||
This chart has been deprecated. Please use the official chart hosted at https://blakeblackshear.github.io/blakeshome-charts/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest changing this line to:
This chart has been deprecated. Please use the official chart hosted at https://github.com/blakeblackshear/blakeshome-charts/tree/master/charts/frigate.
charts/frigate/templates/NOTES.txt
Outdated
echo "Visit http://127.0.0.1:5000 to use your application" | ||
kubectl port-forward $POD_NAME 5000:5000 | ||
{{- end }} | ||
This chart has been deprecated. Please use the official chart hosted at https://blakeblackshear.github.io/blakeshome-charts/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest changing this line to:
This chart has been deprecated. Please use the official chart hosted at https://github.com/blakeblackshear/blakeshome-charts/tree/master/charts/frigate.
Sure thing, will modify them in a bit |
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for making these changes @bjw-s!
Description of the change
Since the chart is now hosted by the app author, this chart can be deprecated.
Benefits
Possible drawbacks
Applicable issues
Additional information
Checklist
Chart.yaml
according to semver.[home-assistant]
)