[PR #1337] [MERGED] move redis only command to if statement in kernel #1332

Closed
opened 2026-02-04 19:37:30 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1337
Author: @notAreYouScared
Created: 5/4/2025
Status: Merged
Merged: 5/5/2025
Merged by: @notAreYouScared

Base: mainHead: charles/updatekernel


📝 Commits (1)

  • 3670fab move redis only command to if statement

📊 Changes

1 file changed (+5 additions, -2 deletions)

View changed files

📝 app/Console/Kernel.php (+5 -2)

📄 Description

Kernel will run this command no matter the setting in the env, per docs, this only applies to cache when it's using redis as the driver, added if statement, no point in having an extra command to run when it will just error out.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/panel/pull/1337 **Author:** [@notAreYouScared](https://github.com/notAreYouScared) **Created:** 5/4/2025 **Status:** ✅ Merged **Merged:** 5/5/2025 **Merged by:** [@notAreYouScared](https://github.com/notAreYouScared) **Base:** `main` ← **Head:** `charles/updatekernel` --- ### 📝 Commits (1) - [`3670fab`](https://github.com/pelican-dev/panel/commit/3670fab00080e78961fbb44aa75fd933b01dc665) move redis only command to if statement ### 📊 Changes **1 file changed** (+5 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `app/Console/Kernel.php` (+5 -2) </details> ### 📄 Description Kernel will run this command no matter the setting in the env, per docs, this only applies to cache when it's using redis as the driver, added if statement, no point in having an extra command to run when it will just error out. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-04 19:37:30 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1332