Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating role to use 1.9.2 #10

Closed
wants to merge 12 commits into from
Closed

Conversation

SiM22
Copy link
Contributor

@SiM22 SiM22 commented Sep 28, 2022

Changes

Improvement: Added pipenv task as per docker build
Improvement: Added new packages
Bugfix: Restart services if gunicorn files changed

Tests: changed prior release to 1.8.0

Copy link
Contributor Author

@SiM22 SiM22 left a comment

Choose a reason for hiding this comment

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

Updating to pass lint tests

@SiM22
Copy link
Contributor Author

SiM22 commented Sep 28, 2022

Tested as working on my own build and passing tests on CI


- name: install paperlessng requirements
- name: Create requirments.txt file
ansible.builtin.shell: ".venv/bin/pipenv requirements > requirements.txt"
Copy link
Member

Choose a reason for hiding this comment

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

Maybe this should check if the requirements.txt exists already? An install from the release archive would, but a source install wouldn't?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll add something in later on tonight

@stumpylog
Copy link
Member

@SiM22 any updates on this PR?

@SiM22
Copy link
Contributor Author

SiM22 commented Jan 31, 2023

@SiM22 any updates on this PR?

none yet, apologies. My son was born in September so i have not had any time these last few months. Will try and get a look into this

@Tronde
Copy link

Tronde commented Feb 10, 2023

Hello @SiM22 and congratulations on being a father!
In case you'll find the time to look into this, please be aware that it doesn't fix #9 as mentioned there.

@SiM22 SiM22 closed this Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants