Move bookmarks startup.py over to AppConfig::ready

cms/envs/common.py

@@ -939,7 +939,7 @@ INSTALLED_APPS = [
     'openedx.core.djangoapps.service_status',
 
     # Bookmarks
-    'openedx.core.djangoapps.bookmarks',
+    'openedx.core.djangoapps.bookmarks.apps.BookmarksConfig',
 
     # Video module configs (This will be moved to Video once it becomes an XBlock)
     'openedx.core.djangoapps.video_config',

lms/envs/common.py

@@ -2076,7 +2076,7 @@ INSTALLED_APPS = [
     'openedx.core.djangoapps.video_config',
 
     # Bookmarks
-    'openedx.core.djangoapps.bookmarks',
+    'openedx.core.djangoapps.bookmarks.apps.BookmarksConfig',
 
     # Our courseware
     'courseware',

openedx/core/djangoapps/bookmarks/apps.py

+"""
+Configuration for bookmarks Django app
+"""
+from django.apps import AppConfig
+
+
+class BookmarksConfig(AppConfig):
+    """
+    Configuration class for bookmarks Django app
+    """
+    name = 'openedx.core.djangoapps.bookmarks'
+    verbose_name = "Bookmarks"
+
+    def ready(self):
+        # Register the signals handled by bookmarks.
+        from . import signals

openedx/core/djangoapps/bookmarks/startup.py

-"""
-Setup the signals on startup.
-"""
-
-from . import signals  # pylint: disable=unused-import