[firefly-iii] Added firefly-iii chart

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 623 forks source link

[firefly-iii] Added firefly-iii chart #1576

Closed bikram990 closed 2 years ago

bikram990 commented 2 years ago

Description of the change

Add firefly-iii chart

Benefits

People will be able to deploy firefly-iii and manage their personal finance.

Possible drawbacks

Applicable issues

Additional information

Checklist

[X] Title of the PR starts with chart name (e.g. [home-assistant])
[X] Chart version bumped in Chart.yaml according to semver.
[X] Chart artifacthub.io/changes changelog annotation has been updated in Chart.yaml. See Artifact Hub documentation for more info.
[x] Variables have been documented in the values.yaml file.

truxnell commented 2 years ago

Hi @bikram990, thanks for the PR Looks like you have some whitespace/end of file linting to do. If you haven't, we suggest you install the repo pre-commit to ensure your PR will past the linting check.

bikram990 commented 2 years ago

@Truxnell I've fixed the pre-commit linting issue.

truxnell commented 2 years ago

@bikram990 I've approved & merged.
Have a look at how the readme generates - you can clean up anything that might need updating in a update (as it usually takes a bit to shake out changes with a new chart)

truxnell commented 2 years ago

@all-contributors please add @bikram990 for code

allcontributors[bot] commented 2 years ago

@Truxnell

I've put up a pull request to add @bikram990! :tada: