k8s-at-home / charts

⚠️ Deprecated : Helm charts for applications you run at home
https://docs.k8s-at-home.com
Apache License 2.0
1.45k stars 621 forks source link

[jellyfin] Update to 10.8.1 and fix typo in Chart.yaml homepage URL #1721

Closed mcronce closed 2 years ago

mcronce commented 2 years ago

Description of the change

Benefits

Possible drawbacks N/A

Applicable issues N/A

Additional information Not sure if 10.7.7 to 10.8.1 would warrant bumping the chart minor version instead of patch, can change it if so

Checklist

mcronce commented 2 years ago

@onedr0p @bjw-s @billimek @carpenike @Truxnell sorry for the @ spam. It looks like you're all intended to be code owners, but .github/CODEOWNERS was renamed to .github/values.yaml in 2a111668 - which I'm guessing was not intentional?

bjw-s commented 2 years ago

@onedr0p @bjw-s @billimek @carpenike @Truxnell sorry for the @ spam. It looks like you're all intended to be code owners, but .github/CODEOWNERS was renamed to .github/values.yaml in 2a11166 - which I'm guessing was not intentional?

Thanks for pointing this out! I've fixed it in the master branch. I'll try to take a look at the PR later this evening / tomorrow

mcronce commented 2 years ago

@bjw-s sorry, bad timing ;) I just pushed a fix for the merge conflicts

bjw-s commented 2 years ago

@bjw-s sorry, bad timing ;) I just pushed a fix for the merge conflicts

haha np :)