Skip to content

Conversation

@fty4
Copy link
Contributor

@fty4 fty4 commented Nov 30, 2024

What this PR does / why we need it

I'd love to see if instead of the image.tag the appVersion is used to set the application version of the chart.

Therefore I removed the tag-value content to use the default.
This also required to add the v prefix to the appVersion.

Another change is to update to the latest linkwarden image v2.8.3.

Special notes for your reviewer

I hope you also like that stlye - otherwise feel free to close this PR.

Updated chart version to 0.3.5 to ensure faster merging after #26.

Checklist

  • Chart Version bumped
  • User name added to AUTHORS file
  • Title of the PR starts with a valid commit scope as detailed in the CONTRIBUTING (e.g.
    fix(charts/linkwarden): ...)

@fty4 fty4 requested a review from mvprowess as a code owner November 30, 2024 17:44
@github-actions
Copy link

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Dec 30, 2024
@github-actions
Copy link

false

@github-actions github-actions bot closed this Feb 28, 2025
@fty4
Copy link
Contributor Author

fty4 commented Mar 23, 2025

@mvprowess this is still an open issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant