Skip to content

Update stirling-pdf to version v0.36.5#1974

Merged
nmfretz merged 1 commit intomasterfrom
update-stirling-pdf-v0.36.5
Dec 30, 2024
Merged

Update stirling-pdf to version v0.36.5#1974
nmfretz merged 1 commit intomasterfrom
update-stirling-pdf-v0.36.5

Conversation

@nmfretz
Copy link
Copy Markdown
Member

@nmfretz nmfretz commented Dec 21, 2024

🤖 This is an automated pull request to update stirling-pdf to version v0.36.5. This PR must be reviewed and tested before merging.

@github-actions
Copy link
Copy Markdown

🎉   Linting finished with no errors or warnings   🎉

Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store.

Please review the linting results below and make any necessary changes to your submission.

Linting Results

Severity File Description
ℹ️ stirling-pdf/docker-compose.yml Potentially using unsafe user in service "app":
The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000.

Legend

Symbol Description
Error: This must be resolved before this PR can be merged.
⚠️ Warning: This is highly encouraged to be resolved, but is not strictly mandatory.
ℹ️ Info: This is just for your information.

@nmfretz
Copy link
Copy Markdown
Member Author

nmfretz commented Dec 21, 2024

🤖 This is an automated summary of installation tests for stirling-pdf:

  • ✅ App successfully installed in umbrel-dev instance
  • ✅ App successfully listed in umbrel.yaml
  • ✅ App UI successfully returns a 200 status code
  • 📸 Screenshot of the app:
    App Screenshot

This PR must still be reviewed and tested before merging.

@nmfretz
Copy link
Copy Markdown
Member Author

nmfretz commented Dec 30, 2024

Reviewed and tested (app update and fresh install)

@nmfretz nmfretz merged commit 388cf91 into master Dec 30, 2024
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.

1 participant