Merge remote-tracking branch 'upstream/develop' into staging

* upstream/develop: fix: don't allow uninstall frappe... Signed-off-by: Akhil Narang <[email protected]>
frappe · Aug 30, 2024 · 225e43b · 225e43b
2 parents a1b19cd + bb2e324
commit 225e43b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/bench/bench.py b/bench/bench.py
@@ -10,7 +10,7 @@
 
 # imports - module imports
 import bench
-from bench.exceptions import AppNotInstalledError, InvalidRemoteException
+from bench.exceptions import AppNotInstalledError, InvalidRemoteException, ValidationError
 from bench.config.common_site_config import setup_config
 from bench.utils import (
 	UNSET_ARG,
@@ -122,6 +122,8 @@ def install(self, app, branch=None):
 		self.apps.sync()
 
 	def uninstall(self, app, no_backup=False, force=False):
+		if app == "frappe":
+			raise ValidationError("You cannot uninstall the app `frappe`")
 		from bench.app import App
 
 		if not force: