search

linuxserver / docker-snipe-it

Alpine/Nginx container for the Asset Management software Snipe-IT
GNU General Public License v3.0
123 stars 26 forks source link

Bump to 3.15 #41

Closed Roxedus closed 2 years ago

Roxedus commented 2 years ago

linuxserver.io

Description:

Benefits of this PR and context:

How Has This Been Tested?

Source / References:

LinuxServer-CI commented 2 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.0-pkg-1ea0fae0-pr-41/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.0-pkg-1ea0fae0-pr-41/shellcheck-result.xml

UnlimitedCookies commented 2 years ago

Good to see the upgrade to PHP 8 finally, as #28 has been stale for some time. Hoping the upgrade to snipe-it v6 follows soon! - Thanks!

nemchik commented 2 years ago

Good to see the upgrade to PHP 8 finally, as #28 has been stale for some time. Hoping the upgrade to snipe-it v6 follows soon! - Thanks!

Progress on #28 is blocked by changes required in the nginx base image and swag. Although this image does not rely on swag, we need to make sure that the nginx base includes everything required for all downstream images (like this one and swag) to properly transition to the new config structure. php8 happened to be lumped in with the structure changes.

It's certainly overdue, but not forgotten. Good progress has been made this week, and hopefully we can continue that trend and get all of the downstream images listed in https://github.com/linuxserver/docker-swag/pull/169 merged to the new nginx base soon.

github-actions[bot] commented 2 years ago

This pull request 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.