Skip to content

Commit 83165b7

Browse files
committed
Fix linting based on new configs
1 parent 9a84b07 commit 83165b7

File tree

1 file changed

+30
-25
lines changed

1 file changed

+30
-25
lines changed

readthedocs/projects/tasks.py

+30-25
Original file line numberDiff line numberDiff line change
@@ -752,11 +752,12 @@ def store_build_artifacts(
752752
"""
753753
if settings.RTD_BUILD_MEDIA_STORAGE:
754754
log.info(
755-
LOG_TEMPLATE.format(
756-
project=self.version.project.slug,
757-
version=self.version.slug,
758-
msg='Writing build artifacts to media storage',
759-
),
755+
LOG_TEMPLATE,
756+
{
757+
'project': self.version.project.slug,
758+
'version': self.version.slug,
759+
'msg': 'Writing build artifacts to media storage',
760+
},
760761
)
761762

762763
storage = get_storage_class(settings.RTD_BUILD_MEDIA_STORAGE)()
@@ -798,23 +799,25 @@ def store_build_artifacts(
798799
include_file=False,
799800
)
800801
log.info(
801-
LOG_TEMPLATE.format(
802-
project=self.version.project.slug,
803-
version=self.version.slug,
804-
msg=f'Writing {media_type} to media storage - {to_path}',
805-
),
802+
LOG_TEMPLATE,
803+
{
804+
'project': self.version.project.slug,
805+
'version': self.version.slug,
806+
'msg': f'Writing {media_type} to media storage - {to_path}',
807+
},
806808
)
807809
try:
808810
storage.copy_directory(from_path, to_path)
809811
except Exception:
810812
# Ideally this should just be an IOError
811813
# but some storage backends unfortunately throw other errors
812814
log.exception(
813-
LOG_TEMPLATE.format(
814-
project=self.version.project.slug,
815-
version=self.version.slug,
816-
msg=f'Error copying {from_path} to storage (not failing build)',
817-
),
815+
LOG_TEMPLATE,
816+
{
817+
'project': self.version.project.slug,
818+
'version': self.version.slug,
819+
'msg': f'Error copying {from_path} to storage (not failing build)',
820+
},
818821
)
819822

820823
for media_type in types_to_delete:
@@ -824,23 +827,25 @@ def store_build_artifacts(
824827
include_file=False,
825828
)
826829
log.info(
827-
LOG_TEMPLATE.format(
828-
project=self.version.project.slug,
829-
version=self.version.slug,
830-
msg=f'Deleting {media_type} from media storage - {media_path}',
831-
),
830+
LOG_TEMPLATE,
831+
{
832+
'project': self.version.project.slug,
833+
'version': self.version.slug,
834+
'msg': f'Deleting {media_type} from media storage - {media_path}',
835+
},
832836
)
833837
try:
834838
storage.delete_directory(media_path)
835839
except Exception:
836840
# Ideally this should just be an IOError
837841
# but some storage backends unfortunately throw other errors
838842
log.exception(
839-
LOG_TEMPLATE.format(
840-
project=self.version.project.slug,
841-
version=self.version.slug,
842-
msg=f'Error deleting {media_path} from storage (not failing build)',
843-
),
843+
LOG_TEMPLATE,
844+
{
845+
'project': self.version.project.slug,
846+
'version': self.version.slug,
847+
'msg': f'Error deleting {media_path} from storage (not failing build)',
848+
},
844849
)
845850

846851
def update_app_instances(

0 commit comments

Comments
 (0)