From 0e8de29b11eaadf3459fb5ce2c05e28c101f316e Mon Sep 17 00:00:00 2001 From: Allan Beaufour Date: Sat, 24 Jan 2026 18:40:55 -0500 Subject: [PATCH 1/2] Test CI: Add intentional lint errors Adding unused imports and long line to verify GitHub Actions CI catches errors. Co-Authored-By: Claude Opus 4.5 --- flickr_api/cache.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/flickr_api/cache.py b/flickr_api/cache.py index 061ac8d..0775b6a 100644 --- a/flickr_api/cache.py +++ b/flickr_api/cache.py @@ -41,6 +41,8 @@ import threading import time +import os +import sys from typing import Any, Callable, TypeVar F = TypeVar("F", bound=Callable[..., Any]) @@ -49,7 +51,7 @@ class SimpleCache(object): """Simple response cache for FlickrAPI calls. - This stores max 50 entries, timing them out after 120 seconds: + This stores max 50 entries, timing them out after 120 seconds. This is a very long line that intentionally exceeds the maximum line length allowed by flake8 which is typically 88 or 100 characters depending on configuration. >>> cache = SimpleCache(timeout=120, max_entries=50) """ From 8a9b14340793e41ef56a6ef06d7c6836133f85cc Mon Sep 17 00:00:00 2001 From: Allan Beaufour Date: Sat, 24 Jan 2026 18:42:40 -0500 Subject: [PATCH 2/2] Test CI: Fix lint, break a test instead - Removed intentional lint errors from cache.py - Added intentional test failure to verify CI catches test failures Co-Authored-By: Claude Opus 4.5 --- flickr_api/cache.py | 4 +--- test/test_parse_sizes.py | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/flickr_api/cache.py b/flickr_api/cache.py index 0775b6a..061ac8d 100644 --- a/flickr_api/cache.py +++ b/flickr_api/cache.py @@ -41,8 +41,6 @@ import threading import time -import os -import sys from typing import Any, Callable, TypeVar F = TypeVar("F", bound=Callable[..., Any]) @@ -51,7 +49,7 @@ class SimpleCache(object): """Simple response cache for FlickrAPI calls. - This stores max 50 entries, timing them out after 120 seconds. This is a very long line that intentionally exceeds the maximum line length allowed by flake8 which is typically 88 or 100 characters depending on configuration. + This stores max 50 entries, timing them out after 120 seconds: >>> cache = SimpleCache(timeout=120, max_entries=50) """ diff --git a/test/test_parse_sizes.py b/test/test_parse_sizes.py index b37a3c3..885e277 100644 --- a/test/test_parse_sizes.py +++ b/test/test_parse_sizes.py @@ -24,7 +24,7 @@ def test_video_largest_size(self): height=2000) }, media="video") - self.assertEqual("HD MP4", p._getLargestSizeLabel()) + self.assertEqual("BROKEN TEST", p._getLargestSizeLabel()) def test_video_none_entry(self): p = f.objects.Photo(