chore: Add typing

frappe/config/__init__.py

@@ -8,10 +8,8 @@ from frappe.desk.moduleview import (
 )
 
 
-def get_modules_from_all_apps_for_user(user=None):
-	if not user:
-		user = frappe.session.user
-
+def get_modules_from_all_apps_for_user(user: str = None) -> list[dict]:
+	user = user or frappe.session.user
 	all_modules = get_modules_from_all_apps()
 	global_blocked_modules = frappe.get_doc("User", "Administrator").get_blocked_modules()
 	user_blocked_modules = frappe.get_doc("User", user).get_blocked_modules()
@@ -61,7 +59,7 @@ def get_all_empty_tables_by_module():
 	empty_tables_by_module = {}
 
 	for doctype, module in results:
-		if "tab" + doctype in empty_tables:
+		if f"tab{doctype}" in empty_tables:
 			if module in empty_tables_by_module:
 				empty_tables_by_module[module].append(doctype)
 			else:

frappe/core/notifications.py

@@ -32,11 +32,10 @@ def get_things_todo(as_list=False):
 
 	if as_list:
 		return data
-	else:
-		return data[0][0]
+	return data[0][0]
 
 
-def get_todays_events(as_list=False):
+def get_todays_events(as_list: bool = False):
 	"""Returns a count of todays events in calendar"""
 	from frappe.desk.doctype.event.event import get_events
 	from frappe.utils import nowdate

frappe/desk/doctype/event/event.py

@@ -205,7 +205,7 @@ def send_event_digest():
 
 
 @frappe.whitelist()
-def get_events(start, end, user=None, for_reminder=False, filters=None):
+def get_events(start, end, user=None, for_reminder=False, filters=None) -> list[frappe._dict]:
 	if not user:
 		user = frappe.session.user