From b42d1c69940264be317e0ccd520f259959108b36 Mon Sep 17 00:00:00 2001
From: farhan <farhan.khan@arbisoft.com>
Date: Fri, 23 Jan 2026 18:59:20 +0500
Subject: [PATCH 1/5] chore: remove obsoleted test case

---
 xblocks_contrib/video/tests/test_video.py | 26 -----------------------
 1 file changed, 26 deletions(-)
 delete mode 100644 xblocks_contrib/video/tests/test_video.py

diff --git a/xblocks_contrib/video/tests/test_video.py b/xblocks_contrib/video/tests/test_video.py
deleted file mode 100644
index 784b92f2..00000000
--- a/xblocks_contrib/video/tests/test_video.py
+++ /dev/null
@@ -1,26 +0,0 @@
-"""
-Tests for VideoBlock
-"""
-
-from django.test import TestCase
-from xblock.fields import ScopeIds
-from xblock.test.toy_runtime import ToyRuntime
-
-from xblocks_contrib import VideoBlock
-
-
-class TestVideoBlock(TestCase):
-    """Tests for VideoBlock"""
-
-    def test_my_student_view(self):
-        """Test the basic view loads."""
-        scope_ids = ScopeIds("1", "2", "3", "4")
-        block = VideoBlock(ToyRuntime(), scope_ids=scope_ids)
-        frag = block.student_view()
-        as_dict = frag.to_dict()
-        content = as_dict["content"]
-        self.assertIn(
-            "VideoBlock: count is now",
-            content,
-            "XBlock did not render correct student view",
-        )

From 38be3fce38a6e37f89c5b4d4f2c9d08706713220 Mon Sep 17 00:00:00 2001
From: farhan <farhan.khan@arbisoft.com>
Date: Fri, 23 Jan 2026 19:15:13 +0500
Subject: [PATCH 2/5] fix: tox docs fixes

---
 docs/conf.py                            |  1 +
 xblocks_contrib/video/tests/__init__.py |  3 +++
 xblocks_contrib/video/video.py          | 23 ++++++++++++++---------
 xblocks_contrib/video/video_handlers.py | 24 +++++++++++++++---------
 xblocks_contrib/video/video_utils.py    |  8 ++++----
 5 files changed, 37 insertions(+), 22 deletions(-)
 create mode 100644 xblocks_contrib/video/tests/__init__.py

diff --git a/docs/conf.py b/docs/conf.py
index 17c0f81c..1a5a43e9 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -80,6 +80,7 @@ def get_version(*file_paths):
 # A list of warning types to suppress arbitrary warning messages.
 suppress_warnings = [
     "image.nonlocal_uri",
+    "intersphinx.external",  # Ignore network failures when fetching intersphinx inventories
 ]
 
 # Add any paths that contain templates here, relative to this directory.
diff --git a/xblocks_contrib/video/tests/__init__.py b/xblocks_contrib/video/tests/__init__.py
new file mode 100644
index 00000000..6fa36e42
--- /dev/null
+++ b/xblocks_contrib/video/tests/__init__.py
@@ -0,0 +1,3 @@
+"""
+Tests for the video xblock.
+"""
diff --git a/xblocks_contrib/video/video.py b/xblocks_contrib/video/video.py
index d1a21813..b7f475d8 100644
--- a/xblocks_contrib/video/video.py
+++ b/xblocks_contrib/video/video.py
@@ -1,12 +1,16 @@
-"""Video is ungraded Xmodule for support video content.
+"""
+Video is ungraded Xmodule for support video content.
 It's new improved video block, which support additional feature:
-- Can play non-YouTube video sources via in-browser HTML5 video player.
-- YouTube defaults to HTML5 mode from the start.
-- Speed changes in both YouTube and non-YouTube videos happen via
-in-browser HTML5 video method (when in HTML5 mode).
-- Navigational subtitles can be disabled altogether via an attribute
-in XML.
-Examples of html5 videos for manual testing:
+
+* Can play non-YouTube video sources via in-browser HTML5 video player.
+* YouTube defaults to HTML5 mode from the start.
+* Speed changes in both YouTube and non-YouTube videos happen via
+  in-browser HTML5 video method (when in HTML5 mode).
+* Navigational subtitles can be disabled altogether via an attribute
+  in XML.
+
+Examples of html5 videos for manual testing::
+
     https://s3.amazonaws.com/edx-course-videos/edx-intro/edX-FA12-cware-1_100.mp4
     https://s3.amazonaws.com/edx-course-videos/edx-intro/edX-FA12-cware-1_100.webm
     https://s3.amazonaws.com/edx-course-videos/edx-intro/edX-FA12-cware-1_100.ogv
@@ -108,7 +112,8 @@ class VideoBlock(
     AjaxHandlerMixin, StudioMetadataMixin,
     LicenseMixin):
     """
-    XML source example:
+    XML source example::
+
         <video show_captions="true"
             youtube="0.75:jNCf2gIqpeE,1.0:ZwkTiUPN0mg,1.25:rsq9auxASqI,1.50:kMyNdzVHHgg"
             url_name="lecture_21_3" display_name="S19V3: Vacancies"
diff --git a/xblocks_contrib/video/video_handlers.py b/xblocks_contrib/video/video_handlers.py
index 3d364aee..0b59a5d9 100644
--- a/xblocks_contrib/video/video_handlers.py
+++ b/xblocks_contrib/video/video_handlers.py
@@ -226,23 +226,26 @@ def transcript(self, request, dispatch):
         """
         Entry point for transcript handlers for student_view.
 
-        Request GET contains:
+        Request GET contains::
+
             (optional) `videoId` for `translation` dispatch.
             `is_bumper=1` flag for bumper case.
 
-        Dispatches, (HTTP GET):
+        Dispatches, (HTTP GET)::
+
             /translation/[language_id]
             /download
             /available_translations/
 
-        Explanations:
+        Explanations::
+
             `download`: returns SRT or TXT file.
             `translation`: depends on HTTP methods:
                     Provide translation for requested language, SJSON format is sent back on success,
                     Proper language_id should be in url.
             `available_translations`:
                     Returns list of languages, for which transcript files exist.
-                    For 'en' check if SJSON exists. For non-`en` check if SRT file exists.
+                    For 'en' check if SJSON exists. For non-'en' check if SRT file exists.
         """
         is_bumper = request.GET.get('is_bumper', False)
         transcripts = self.get_transcripts_info(is_bumper)
@@ -410,17 +413,20 @@ def studio_transcript(self, request, dispatch):
         """
         Entry point for Studio transcript handlers.
 
-        Dispatches:
-            /translation/[language_id] - language_id sould be in url.
+        Dispatches::
+
+            /translation/[language_id] - language_id should be in url.
+
+        ``translation`` dispatch support following HTTP methods::
 
-        `translation` dispatch support following HTTP methods:
             `POST`:
                 Upload srt file. Check possibility of generation of proper sjson files.
                 For now, it works only for self.transcripts, not for `en`.
-            `GET:
+            `GET`:
                 Return filename from storage. SRT format is sent back on success. Filename should be in GET dict.
 
-        We raise all exceptions right in Studio:
+        We raise all exceptions right in Studio::
+
             TranscriptNotFoundError:
                 Video or asset was deleted from module/contentstore, but request came later.
                 Seems impossible to be raised. block_render.py catches NotFoundErrors from here.
diff --git a/xblocks_contrib/video/video_utils.py b/xblocks_contrib/video/video_utils.py
index 3695d20f..65ec9453 100644
--- a/xblocks_contrib/video/video_utils.py
+++ b/xblocks_contrib/video/video_utils.py
@@ -48,10 +48,10 @@ def rewrite_video_url(cdn_base_url, original_video_url):
     Re-write rules for country codes are specified via the
     EDX_VIDEO_CDN_URLS configuration structure.
 
-    :param cdn_base_url: The scheme, hostname, port and any relevant path prefix for the alternate CDN,
-    for example: https://mirror.example.cn/edx
-    :param original_video_url: The canonical source for this video, for example:
-    https://cdn.example.com/edx-course-videos/VIDEO101/001.mp4
+    :param cdn_base_url: The scheme, hostname, port and any relevant path prefix
+        for the alternate CDN, for example: https://mirror.example.cn/edx
+    :param original_video_url: The canonical source for this video,
+        for example: https://cdn.example.com/edx-course-videos/VIDEO101/001.mp4
     :return: The re-written URL
     """
 

From a192a444aab4a2a8a58345876249523feaea187e Mon Sep 17 00:00:00 2001
From: farhan <farhan.khan@arbisoft.com>
Date: Fri, 23 Jan 2026 19:55:57 +0500
Subject: [PATCH 3/5] fix: tox quality fixes

---
 xblocks_contrib/html/html.py                  |  6 +--
 xblocks_contrib/video/content.py              | 21 ++++++---
 xblocks_contrib/video/mixin.py                |  6 ++-
 .../video/studio_metadata_mixin.py            |  6 ++-
 xblocks_contrib/video/validation.py           |  4 +-
 xblocks_contrib/video/video.py                | 45 ++++++++++---------
 .../video/video_transcripts_utils.py          |  8 +++-
 xblocks_contrib/video/video_utils.py          | 20 ++++++---
 xblocks_contrib/video/video_xfields.py        | 11 ++---
 9 files changed, 79 insertions(+), 48 deletions(-)

diff --git a/xblocks_contrib/html/html.py b/xblocks_contrib/html/html.py
index 282e6292..7d3c6301 100644
--- a/xblocks_contrib/html/html.py
+++ b/xblocks_contrib/html/html.py
@@ -322,7 +322,7 @@ def get_context(self):
         # the root /c4x/ url for assets. This allows client-side substitutions to occur.
         return {
             "module": self,
-            "editable_metadata_fields": self.editable_metadata_fields,  # pylint: disable=no-member
+            "editable_metadata_fields": self.editable_metadata_fields,
             "data": self.data,
             "base_asset_url": self.get_base_url_path_for_course_assets(self.location.course_key),
             "enable_latex_compiler": self.use_latex_compiler,
@@ -371,7 +371,7 @@ def bind_for_student(self, user_id, wrappers=None):
         if self.scope_ids.user_id is not None and user_id == self.scope_ids.user_id:
             if getattr(self.runtime, "position", None):
                 # update the position of the tab
-                self.position = self.runtime.position  # pylint: disable=attribute-defined-outside-init
+                self.position = self.runtime.position
             return
 
         # # If we are switching users mid-request, save the data from the old user.
@@ -402,7 +402,7 @@ def bind_for_student(self, user_id, wrappers=None):
             wrapped_field_data = self.runtime.service(self, "field-data-unbound")
             for wrapper in wrappers:
                 wrapped_field_data = wrapper(wrapped_field_data)
-            self._bound_field_data = wrapped_field_data  # pylint: disable=attribute-defined-outside-init
+            self._bound_field_data = wrapped_field_data
             if getattr(self.runtime, "uses_deprecated_field_data", False):
                 # This approach is deprecated but old mongo's CachingDescriptorSystem still requires it.
                 # For Split mongo's CachingDescriptor system, don't set ._field_data this way.
diff --git a/xblocks_contrib/video/content.py b/xblocks_contrib/video/content.py
index 461e278e..0259aea5 100644
--- a/xblocks_contrib/video/content.py
+++ b/xblocks_contrib/video/content.py
@@ -1,13 +1,21 @@
-""" Video Block Static Content, class copied from StaticContent class in edx-platform/xmodule/contentstore/content.py """
+"""
+Video Block Static Content.
+
+Class copied from StaticContent class in edx-platform/xmodule/contentstore/content.py
+"""
 from opaque_keys import InvalidKeyError
 from opaque_keys.edx.keys import AssetKey
 from opaque_keys.edx.locator import AssetLocator
 
-class VideoBlockStaticContent:  # lint-amnesty, pylint: disable=missing-class-docstring
-    def __init__(self, loc, name, content_type, data, last_modified_at=None, thumbnail_location=None, import_path=None,
-                 length=None, locked=False, content_digest=None):
+
+class VideoBlockStaticContent:  # pylint: disable=missing-class-docstring
+    def __init__(  # pylint: disable=too-many-positional-arguments
+        self, loc, name, content_type, data, last_modified_at=None,
+        thumbnail_location=None, import_path=None, length=None, locked=False, content_digest=None
+    ):
         self.location = loc
-        self.name = name  # a display string which can be edited, and thus not part of the location which needs to be fixed  # lint-amnesty, pylint: disable=line-too-long
+        # a display string which can be edited, and thus not part of the location which needs to be fixed
+        self.name = name
         self.content_type = content_type
         self._data = data
         self.length = length
@@ -20,7 +28,7 @@ def __init__(self, loc, name, content_type, data, last_modified_at=None, thumbna
         self.content_digest = content_digest
 
     @staticmethod
-    def compute_location(course_key, path, revision=None, is_thumbnail=False):  # lint-amnesty, pylint: disable=unused-argument
+    def compute_location(course_key, path, revision=None, is_thumbnail=False):  # pylint: disable=unused-argument
         """
         Constructs a location object for static content.
 
@@ -48,6 +56,7 @@ def get_location_from_path(path):
             if path.startswith('/') or path.endswith('/'):
                 # try stripping off the leading slash and try again
                 return AssetKey.from_string(path.strip('/'))
+            return None
 
     @staticmethod
     def serialize_asset_key_with_slash(asset_key):
diff --git a/xblocks_contrib/video/mixin.py b/xblocks_contrib/video/mixin.py
index b75b466e..db4bc789 100644
--- a/xblocks_contrib/video/mixin.py
+++ b/xblocks_contrib/video/mixin.py
@@ -5,9 +5,11 @@
 from xblock.core import XBlockMixin
 from xblock.fields import Scope, String
 
-# Make '_' a no-op so we can scrape strings. Using lambda instead of
+
+# Make '_' a no-op so we can scrape strings. Using dummy function instead of
 #  `django.utils.translation.gettext_noop` because Django cannot be imported in this file
-_ = lambda text: text
+def _(text):
+    return text
 
 
 class LicenseMixin(XBlockMixin):
diff --git a/xblocks_contrib/video/studio_metadata_mixin.py b/xblocks_contrib/video/studio_metadata_mixin.py
index 1c3808c3..38fd9345 100644
--- a/xblocks_contrib/video/studio_metadata_mixin.py
+++ b/xblocks_contrib/video/studio_metadata_mixin.py
@@ -123,7 +123,7 @@ def editable_metadata_fields(self):
         editable_fields.pop('sub')
 
         languages = [{'label': label, 'code': lang} for lang, label in settings.ALL_LANGUAGES]
-        languages.sort(key=lambda l: l['label'])
+        languages.sort(key=lambda lang_item: lang_item['label'])
         editable_fields['transcripts']['custom'] = True
         editable_fields['transcripts']['languages'] = languages
         editable_fields['transcripts']['type'] = 'VideoTranslations'
@@ -148,7 +148,9 @@ def editable_metadata_fields(self):
         if video_config_service:
             for sub_id in possible_sub_ids:
                 try:
-                    _, sub_id, _ = video_config_service.get_transcript(self, lang='en', output_format=TranscriptExtensions.TXT)
+                    _, sub_id, _ = video_config_service.get_transcript(
+                        self, lang='en', output_format=TranscriptExtensions.TXT
+                    )
                     transcripts_info['transcripts'] = dict(transcripts_info['transcripts'], en=sub_id)
                     break
                 except TranscriptNotFoundError:
diff --git a/xblocks_contrib/video/validation.py b/xblocks_contrib/video/validation.py
index 2496f54c..45388ace 100644
--- a/xblocks_contrib/video/validation.py
+++ b/xblocks_contrib/video/validation.py
@@ -15,7 +15,9 @@ class StudioValidationMessage(ValidationMessage):
 
     TYPES = [ValidationMessage.WARNING, ValidationMessage.ERROR, NOT_CONFIGURED]
 
-    def __init__(self, message_type, message_text, action_label=None, action_class=None, action_runtime_event=None):
+    def __init__(  # pylint: disable=too-many-positional-arguments
+        self, message_type, message_text, action_label=None, action_class=None, action_runtime_event=None
+    ):
         """
         Create a new message.
 
diff --git a/xblocks_contrib/video/video.py b/xblocks_contrib/video/video.py
index b7f475d8..4bb79a80 100644
--- a/xblocks_contrib/video/video.py
+++ b/xblocks_contrib/video/video.py
@@ -96,10 +96,6 @@
 log = logging.getLogger(__name__)
 loader = ResourceLoader(__name__)
 
-# Make '_' a no-op so we can scrape strings. Using lambda instead of
-#  `django.utils.translation.ugettext_noop` because Django cannot be imported in this file
-_ = lambda text: text
-
 EXPORT_IMPORT_COURSE_DIR = 'course'
 EXPORT_IMPORT_STATIC_DIR = 'static'
 
@@ -107,10 +103,10 @@
 @XBlock.wants('settings', 'completion', 'request_cache', 'video_config')
 @XBlock.needs('i18n', 'user')
 class VideoBlock(
-    VideoFields, VideoTranscriptsMixin, VideoStudioViewHandlers, VideoStudentViewHandlers,
-    StudioEditableXBlockMixin, LegacyXmlMixin, XBlock,
-    AjaxHandlerMixin, StudioMetadataMixin,
-    LicenseMixin):
+        VideoFields, VideoTranscriptsMixin, VideoStudioViewHandlers, VideoStudentViewHandlers,
+        StudioEditableXBlockMixin, LegacyXmlMixin, XBlock,
+        AjaxHandlerMixin, StudioMetadataMixin,
+        LicenseMixin):
     """
     XML source example::
 
@@ -320,7 +316,7 @@ def public_view(self, context):
         fragment.initialize_js("Video")
         return fragment
 
-    def get_html(self, view=STUDENT_VIEW, context=None):  # lint-amnesty, pylint: disable=arguments-differ, too-many-statements
+    def get_html(self, view=STUDENT_VIEW, context=None):  # pylint: disable=too-many-statements
         """
         Return html for a given view of this block.
         """
@@ -465,7 +461,7 @@ def get_html(self, view=STUDENT_VIEW, context=None):  # lint-amnesty, pylint: di
             'completionEnabled': completion_enabled,
             'completionPercentage': settings.COMPLETION_VIDEO_COMPLETE_PERCENTAGE,
             'duration': video_duration,
-            'end': self.end_time.total_seconds(),  # pylint: disable=no-member
+            'end': self.end_time.total_seconds(),
             'generalSpeed': self.global_speed,
             'lmsRootURL': settings.LMS_ROOT_URL,
             'poster': poster,
@@ -475,7 +471,7 @@ def get_html(self, view=STUDENT_VIEW, context=None):  # lint-amnesty, pylint: di
             # this user, based on what was recorded the last time we saw the
             # user, and defaulting to True.
             'recordedYoutubeIsAvailable': self.youtube_is_available,
-            'savedVideoPosition': self.saved_video_position.total_seconds(),  # pylint: disable=no-member
+            'savedVideoPosition': self.saved_video_position.total_seconds(),
             'saveStateEnabled': not is_public_view,
             'saveStateUrl': self.ajax_url + '/save_user_state',
             # Despite the setting on the block, don't show transcript by default
@@ -483,7 +479,7 @@ def get_html(self, view=STUDENT_VIEW, context=None):  # lint-amnesty, pylint: di
             'showCaptions': json.dumps(self.show_captions and not is_embed),
             'sources': sources,
             'speed': self.speed,
-            'start': self.start_time.total_seconds(),  # pylint: disable=no-member
+            'start': self.start_time.total_seconds(),
             'streams': self.youtube_streams,
             'transcriptAvailableTranslationsUrl': self.runtime.handler_url(
                 self, 'transcript', 'available_translations'
@@ -530,7 +526,8 @@ def get_html(self, view=STUDENT_VIEW, context=None):  # lint-amnesty, pylint: di
             'poster': json.dumps(get_poster(self)),
             'track': track_url,
             'transcript_download_format': transcript_download_format,
-            'transcript_download_formats_list': self.fields['transcript_download_format'].values,  # lint-amnesty, pylint: disable=unsubscriptable-object
+            # pylint: disable=unsubscriptable-object
+            'transcript_download_formats_list': self.fields['transcript_download_format'].values,
             'transcript_feedback_enabled': self.is_transcript_feedback_enabled(),
         }
         video_config_service = self.runtime.service(self, 'video_config')
@@ -631,9 +628,10 @@ def parse_xml_new_runtime(cls, node, runtime, keys):
         video_block = runtime.construct_xblock_from_class(cls, keys)
         field_data = cls.parse_video_xml(node)
         for key, val in field_data.items():
-            if key not in cls.fields:  # lint-amnesty, pylint: disable=unsupported-membership-test
+            # pylint: disable=unsubscriptable-object
+            if key not in cls.fields:  # pylint: disable=unsupported-membership-test
                 continue  # parse_video_xml returns some old non-fields like 'source'
-            setattr(video_block, key, cls.fields[key].from_json(val))  # lint-amnesty, pylint: disable=unsubscriptable-object
+            setattr(video_block, key, cls.fields[key].from_json(val))
         # Don't use VAL in the new runtime:
         video_block.edx_video_id = None
         return video_block
@@ -668,6 +666,7 @@ def parse_xml(cls, node, runtime, _keys):
                 # value["filename"] = definition.get("filename", ["", None])
                 video.xml_attributes.update(value)
             elif field_name in video.fields:
+                # pylint: disable=unsubscriptable-object
                 setattr(video, field_name, cls.fields[field_name].from_json(value))
 
         # Update VAL with info extracted from `node`
@@ -704,7 +703,8 @@ def definition_to_xml(self, resource_fs):  # lint-amnesty, pylint: disable=too-m
             # Mild workaround to ensure that tests pass -- if a field
             # is set to its default value, we don't write it out.
             if value:
-                if key in self.fields and self.fields[key].is_set_on(self):  # lint-amnesty, pylint: disable=unsubscriptable-object, unsupported-membership-test
+                # pylint: disable=unsupported-membership-test,unsubscriptable-object
+                if key in self.fields and self.fields[key].is_set_on(self):
                     try:
                         xml.set(key, str(value))
                     except UnicodeDecodeError:
@@ -969,7 +969,8 @@ def parse_video_xml(cls, xml, id_generator=None):
             else:
                 # We export values with json.dumps (well, except for Strings, but
                 # for about a month we did it for Strings also).
-                field_data[attr] = deserialize_field(cls.fields[attr], value)  # lint-amnesty, pylint: disable=unsubscriptable-object
+                # pylint: disable-next=unsubscriptable-object
+                field_data[attr] = deserialize_field(cls.fields[attr], value)
 
         course_id = getattr(id_generator, 'target_course_id', None)
         # Update the handout location with current course_id
@@ -1085,7 +1086,8 @@ def request_cache(self):
     @request_cached(
         request_cache_getter=lambda args, kwargs: args[1],
     )
-    def get_cached_val_data_for_course(cls, request_cache, video_profile_names, course_id):  # lint-amnesty, pylint: disable=unused-argument
+    # pylint: disable=unused-argument
+    def get_cached_val_data_for_course(cls, request_cache, video_profile_names, course_id):
         """
         Returns the VAL data for the requested video profiles for the given course.
         """
@@ -1204,7 +1206,8 @@ def bind_for_student(self, user_id, wrappers=None):
         # Skip rebinding if we're already bound a user, and it's this user.
         if self.scope_ids.user_id is not None and user_id == self.scope_ids.user_id:
             if getattr(self.runtime, "position", None):
-                self.position = self.runtime.position  # update the position of the tab
+                # update the position of the tab
+                self.position = self.runtime.position  # pylint: disable=attribute-defined-outside-init
             return
 
         # If we are switching users mid-request, save the data from the old user.
@@ -1235,14 +1238,14 @@ def bind_for_student(self, user_id, wrappers=None):
             wrapped_field_data = self.runtime.service(self, "field-data-unbound")
             for wrapper in wrappers:
                 wrapped_field_data = wrapper(wrapped_field_data)
-            self._bound_field_data = wrapped_field_data
+            self._bound_field_data = wrapped_field_data  # pylint: disable=attribute-defined-outside-init
             if getattr(self.runtime, "uses_deprecated_field_data", False):
                 # This approach is deprecated but OldModuleStoreRuntime still requires it.
                 # For SplitModuleStoreRuntime, don't set ._field_data this way.
                 self._field_data = wrapped_field_data
 
     @classmethod
-    def definition_from_xml(cls, xml_object, system):  # lint-amnesty, pylint: disable=unused-argument
+    def definition_from_xml(cls, xml_object, system):
         if len(xml_object) == 0 and len(list(xml_object.items())) == 0:
             return {"data": ""}, []
         return {"data": etree.tostring(xml_object, pretty_print=True, encoding="unicode")}, []
diff --git a/xblocks_contrib/video/video_transcripts_utils.py b/xblocks_contrib/video/video_transcripts_utils.py
index 0a4b9b6a..c7fc58b4 100644
--- a/xblocks_contrib/video/video_transcripts_utils.py
+++ b/xblocks_contrib/video/video_transcripts_utils.py
@@ -46,6 +46,7 @@ def get_html5_ids(html5_sources):
     html5_ids = [x.split('/')[-1].rsplit('.', 1)[0] for x in html5_sources]
     return html5_ids
 
+
 def subs_filename(subs_id, lang='en'):
     """
     Generate proper filename for storage.
@@ -55,6 +56,7 @@ def subs_filename(subs_id, lang='en'):
     else:
         return f'{lang}_subs_{subs_id}.srt.sjson'
 
+
 def clean_video_id(edx_video_id):
     """
     Cleans an edx video ID.
@@ -64,6 +66,7 @@ def clean_video_id(edx_video_id):
     """
     return edx_video_id and edx_video_id.strip()
 
+
 def get_available_transcript_languages(edx_video_id):
     """
     Gets available transcript languages for a video.
@@ -81,6 +84,7 @@ def get_available_transcript_languages(edx_video_id):
 
     return available_languages
 
+
 class VideoTranscriptsMixin:
     """Mixin class for transcript functionality.
 
@@ -163,7 +167,7 @@ def resolve_language_code_to_transcript_code(transcripts, dest_lang):
     """
     Attempts to match the requested dest lang with the existing transcript languages
     """
-    sub, other_lang = transcripts["sub"], transcripts["transcripts"]
+    sub, other_lang = transcripts["sub"], transcripts["transcripts"]  # pylint: disable=unused-variable
     # lang code exists in list of other transcript languages as-is
     if dest_lang in other_lang:
         return dest_lang
@@ -188,6 +192,8 @@ def resolve_language_code_to_transcript_code(transcripts, dest_lang):
             log.debug("language code %s resolved to generic %s", dest_lang, generic_lang_code)
             return generic_lang_code
 
+    return None
+
 
 def get_endonym_or_label(language_code):
     """
diff --git a/xblocks_contrib/video/video_utils.py b/xblocks_contrib/video/video_utils.py
index 65ec9453..fb3a77a1 100644
--- a/xblocks_contrib/video/video_utils.py
+++ b/xblocks_contrib/video/video_utils.py
@@ -85,7 +85,7 @@ def get_poster(video):
     Poster metadata is dict of youtube url for image thumbnail and edx logo
     """
     if not video.bumper.get("enabled"):
-        return
+        return None
 
     poster = OrderedDict({"url": "", "type": ""})
 
@@ -196,14 +196,20 @@ def load_metadata_from_youtube(video_id, request):
                     if res_json.get('items', []):
                         metadata = res_json
                     else:
-                        logging.warning('Unable to find the items in response. Following response '
-                                        'was received: {res}'.format(res=res.text))
+                        logging.warning(
+                            'Unable to find the items in response. Following response was received: %s',
+                            res.text
+                        )
                 except ValueError:
-                    logging.warning('Unable to decode response to json. Following response '
-                                    'was received: {res}'.format(res=res.text))
+                    logging.warning(
+                        'Unable to decode response to json. Following response was received: %s',
+                        res.text
+                    )
             else:
-                logging.warning('YouTube API request failed with status code={status} - '
-                                'Error message is={message}'.format(status=status_code, message=res.text))
+                logging.warning(
+                    'YouTube API request failed with status code=%s - Error message is=%s',
+                    status_code, res.text
+                )
         except (Timeout, ConnectionError):
             logging.warning('YouTube API request failed because of connection time out or connection error')
     else:
diff --git a/xblocks_contrib/video/video_xfields.py b/xblocks_contrib/video/video_xfields.py
index 09d0a40d..c823efe8 100644
--- a/xblocks_contrib/video/video_xfields.py
+++ b/xblocks_contrib/video/video_xfields.py
@@ -7,10 +7,11 @@
 
 from xblock.fields import Boolean, DateTime, Dict, Float, List, RelativeTime, Scope, String
 
-# TODO: Review the following _ function
-# Make '_' a no-op so we can scrape strings. Using lambda instead of
-#  `django.utils.translation.ugettext_noop` because Django cannot be imported in this file
-_ = lambda text: text
+
+# Make '_' a no-op so we can scrape strings. Using dummy function instead of
+#  `django.utils.translation.gettext_noop` because Django cannot be imported in this file
+def _(text):
+    return text
 
 
 class VideoFields:
@@ -75,7 +76,7 @@ class VideoFields:
         scope=Scope.settings,
         default=datetime.timedelta(seconds=0)
     )
-    #front-end code of video player checks logical validity of (start_time, end_time) pair.
+    # front-end code of video player checks logical validity of (start_time, end_time) pair.
 
     download_video = Boolean(
         help=_("Allow students to download versions of this video in different formats if they cannot use the edX video"

From 5224818b877f9084eac4f2a6532b9cd017426b81 Mon Sep 17 00:00:00 2001
From: farhan <farhan.khan@arbisoft.com>
Date: Fri, 23 Jan 2026 20:17:16 +0500
Subject: [PATCH 4/5] chore: remove unwanted line

---
 docs/conf.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/docs/conf.py b/docs/conf.py
index 1a5a43e9..17c0f81c 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -80,7 +80,6 @@ def get_version(*file_paths):
 # A list of warning types to suppress arbitrary warning messages.
 suppress_warnings = [
     "image.nonlocal_uri",
-    "intersphinx.external",  # Ignore network failures when fetching intersphinx inventories
 ]
 
 # Add any paths that contain templates here, relative to this directory.

From 87f8194fda89c5b2def20085a026ec32817f7c68 Mon Sep 17 00:00:00 2001
From: farhan <farhan.khan@arbisoft.com>
Date: Wed, 28 Jan 2026 11:36:11 +0500
Subject: [PATCH 5/5] chore: Adds class doc string

---
 xblocks_contrib/video/content.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/xblocks_contrib/video/content.py b/xblocks_contrib/video/content.py
index 0259aea5..a9f9fa0d 100644
--- a/xblocks_contrib/video/content.py
+++ b/xblocks_contrib/video/content.py
@@ -8,7 +8,10 @@
 from opaque_keys.edx.locator import AssetLocator
 
 
-class VideoBlockStaticContent:  # pylint: disable=missing-class-docstring
+class VideoBlockStaticContent:
+    """
+    Represents static content associated with a video block (such as video files or related assets).
+    """
     def __init__(  # pylint: disable=too-many-positional-arguments
         self, loc, name, content_type, data, last_modified_at=None,
         thumbnail_location=None, import_path=None, length=None, locked=False, content_digest=None