Skip to content
This repository has been archived by the owner on Aug 22, 2022. It is now read-only.

[paperless] chore: switch to paperless-ngx project #1436

Merged
merged 4 commits into from
Mar 4, 2022
Merged

[paperless] chore: switch to paperless-ngx project #1436

merged 4 commits into from
Mar 4, 2022

Conversation

morremeyer
Copy link
Contributor

Description of the change

This switches to the community fork of paperless-ng. The tracking ticket in paperless-ngx is paperless-ngx/paperless-ngx#68.

Please see the following references for the community discussion of the decision to fork:

Benefits

Switch to the community maintained paperless-ngx fork.

Possible drawbacks

None that I am aware of.

Additional information

This is a non-breaking change for the paperless application itself, but as it changes the upstream, I've opted to bump the major version.

Checklist

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

@ghost ghost added the size/XS Categorises a PR that changes 0-9 lines, ignoring generated files. label Feb 24, 2022
@morremeyer morremeyer marked this pull request as draft February 24, 2022 14:17
@morremeyer
Copy link
Contributor Author

Can someone please advise me how to correctly update the README? I can't find documentation for it, although it suspiciously looks like helm-docs.

@ghost ghost added precommit:ok CI status: pre-commit validation successful changelog:ok CI status: changelog validation successful lint:ok CI status: linting successful install:ok CI status: install successful labels Feb 24, 2022
@bjw-s
Copy link
Contributor

bjw-s commented Feb 24, 2022

Can someone please advise me how to correctly update the README? I can't find documentation for it, although it suspiciously looks like helm-docs.

No need to manually update the README, we auto-generate that upon merging the PR. Any manual modifications will be lost then.

I will take a look at the PR later tonight 😃

@morremeyer
Copy link
Contributor Author

@bjw-s that's good to know, thanks! How can I take care of updating the Source Code section then? I didn't find it anywhere else and updated it directly in the README.md for that reason (https://github.com/k8s-at-home/charts/pull/1436/files#diff-2e548bd16ddaf26ec88fbd01ce605997e61c43301e73c5a723aaf90a21908156R11)

@angelnu
Copy link
Contributor

angelnu commented Feb 24, 2022

@morremeyer - that is in at

- https://github.com/jonaswinkler/paperless-ng

As user of paperless and contributor to the k8s-at-home I am really glad to see the project moves on. So no reason for not switching the chart to the new community fork.

@ghost ghost added size/S Categorises a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Categorises a PR that changes 0-9 lines, ignoring generated files. labels Feb 24, 2022
Copy link
Contributor

@bjw-s bjw-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good, thanks for making this PR and moving us to the new fork!
Just a tiny change requested before I merge it.

charts/stable/paperless/values.yaml Outdated Show resolved Hide resolved
@morremeyer morremeyer marked this pull request as ready for review February 25, 2022 18:47
@morremeyer
Copy link
Contributor Author

@bjw-s Thanks for the review. I overlooked that you added it as a suggestion, so I updated it locally.

charts/stable/paperless/Chart.yaml Outdated Show resolved Hide resolved
charts/stable/paperless/Chart.yaml Outdated Show resolved Hide resolved
morremeyer and others added 2 commits February 28, 2022 22:53
Co-authored-by: Bᴇʀɴᴅ Sᴄʜᴏʀɢᴇʀs <[email protected]>
Co-authored-by: Bᴇʀɴᴅ Sᴄʜᴏʀɢᴇʀs <[email protected]>
@bjw-s bjw-s merged commit a377c3d into k8s-at-home:master Mar 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
changelog:ok CI status: changelog validation successful install:ok CI status: install successful lint:ok CI status: linting successful precommit:ok CI status: pre-commit validation successful size/S Categorises a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants